Sounak Mukherjee• 560
@kanuos
Posted
Hi @petronela88, congratulations on your first submission. Your solution looks decent however there's room for improvement. Here's some of the things you should fix and regenerate a new screenshot -
- Convert the
div.container
tomain.container
- that will resolve your landmark issues - Add a
lang="en"
attribute to the HTML - Convert the
h3
tag toh1
- Hide the
footer
and it's contents as they are not there in the design. Hide them using CSS, so that they are available for the screen-readers to read. Look upscreen reader classes
to do so. Here's a link if you want to read more about it. - The entire heading and text should have
text-align: center
as per the design. - Color of the
h3
tag in your code looks a bit darker than the design - Reduce the padding/margin below the
h3
Hope this helps.
Hope to see more of your submissions soon. Happy coding :)
Marked as helpful
1