Szymon Rojek• 4,540
@SzymonRojek
Posted
Hi Vaishnav,
Well done :D
A few tips below:
- instead of div class="header", use the header tag;
- we can use only one the h1 on a page, in this project you can create one h1 with two spans inside main-heading: Reliable, efficient delivery and sub-heading: Powered by Technology;
- instead of div class="container" you can use the main tag just to indicate the main content of a page;
- alt text => In this project, images have an only a decorative role - that's a reason why alt text should be provided as an empty (alt="") so img can be ignored by assistive technologies, such as screen readers;
- div class="card-icon", div class="card card-four" are unnecessary;
- check a project in your browser by the inspector on different devices => what's about the tablet breakpoint (two rows with two boxes each). In my opinion one column till 1200px is too long, check it out;
- add max-width to the container;
Ps. Don't forget to upvote any comments on here that you find helpful.
Greetings :D
0
