Mohamed ELIDRISSI• 455
@elidrissidev
Posted
Good job Dusan! I have a few suggestions for you:
- First I recommend to indent your html properly so that other people can read it easily.
- I also suggest to take a look in "Accessibility Issues", it can help you find issues with your solutions.
- I see you've gone from using an
h1
to using anh3
, skippingh2
. Heading levels help set a content hierarchy for your page, so it is better that you follow them from 1 to 6. - For the "Why Us" section, the content is a list of reasons why choose this fictional service. Since it is a list, I would have used the semantic
ul
element, instead of just a paragraph with line breaks, then you can style it to match the design. - Last thing, I noticed the extra space below the top white section, which can be fixed by removing the
height: 80%
from.container
to make the card's height adjust to fit the content.
Good luck!
Marked as helpful
1
Dušan Lukić• 1,660
@dusanlukic404
Posted
@elidrissidev Thanks Mohamed!
0