@SzymonRojek
Posted
Hi Folarin,
Very nice RWD! :D
I have checked your HTML structure, a few tips for you:
- for this top text I suggest to use the header tag;
- this is a single page component so I think the main content will be below the header => the main tag will be here very useful, what do you think?
- section tag: also possibly there is another solution, main tag as a wrap for all content (header and section below with three divs);
- h1, I don't recommend to use br tag (only when this is really needed but in this solution is not). You can use h1 with two spans inside of it (main-heading and sub-heading);
- alt text => in this project, icons have only decorative role - that's a reason why alt text should be provided as a empty (alt="") so these icons can be ignored by assistive technologies, such as screen readers;
- I would recommend to use min-width instead of max-width;
Generally, good job => keep going! That's it from me.
Ps. Don't forget to upvote any comments on here that you find helpful.
Greetings :D