Responsive page using CSS

Community feedback

• 

  Michel Moreira•300

  @michel-moreiraPosted almost 2 years ago

  Hello Swagnik Das!

  Great job on complete this challenge!

  I have some suggestions tha might be helpful!

  • In the img tag, you must put an alternate text, the attribute "alt". The required alt attribute provides an alternate text for an image, if the user for some reason cannot view it (because of slow connection, an error in the src attribute, or if the user uses a screen reader). Source

  • You must put a h1 tag on the pag cause the page should contain a level-one heading, you could replace the p tag referent to the name and maintain the class the same.

  • All you code should be wrapped by tag landmark, before the tag main and between the tags main and footer you've putted a img to treat a background-image, you could do this in the CSS stylesheet and remove from the HTML document. With background-image you can manipulate in CSS. An article about it.

  • In the class stats you could use a tag <ul> <li><p> </p></li> </ul> with 3 li rather than divs.

  I hope you find it helpful!

  Happy Coding!

  0