Responsive landing page using grid, flex, sass.

Community feedback

• 

  FrontendBy-GJ•430

  @FrontendBy-GJPosted almost 2 years ago

  Great work on the layout! You seem to have forgotten to add alternate text for some of your images. You shouldn't leave them empty, if the image doesn't load properly, the alternate text would then be used telling the user what the image is, like a description almost. Also, your index.html file has some commented code you aren't using like the attributions and the style for that piece of code. If I were you, I would uncomment that code and use it, place it at the bottom (footer). I should be doing the same :)

  0