HTML Markup, Flexbox, Mobile-first workflow

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 3 years ago

  Hey,

  This looks pretty good, well done 👏

  Small things to look at

  • try to make desktop align on the right and left like the design (top and bottom sections line up at the sides)
  • try to center content on mobile, even when in landscape view.
  • remove all the alt text values on those stars. They should be alt=""
  • change the h2s for paragraphs. They have no content under them so shouldn't be headings.
  • consider adding a screenreader only (and SEO) h2 heading before the bottom section. OR swap the h3s in testimonials for h2s. Because headings should go in order.

  I hope this is helpful. Keep on coding

  0

  ---

  

  Lidia•180

  @LidiaRJPosted over 3 years ago

  Hi @grace-snow! Thank you so much for taking the time to send me some feedback. I find your advice very helpful and I will take them into account for my next project. What do you mean with "screenreader only (and SEO) h2 heading"?? I'm still a newbie and get lost with some terms. Thanks again!

  Hope you're having a great Wednesday.

  0