Basic HTML/CSS mobile and desktop versions

Community feedback

• 

  El-Sayed A. Naser•90

  @sayednaserPosted almost 3 years ago

  Great work <3

  I've noticed one issue. You are using <p> tag for all the text. While it visually causes no problem. It causes accessibility issues (e.g for screen readers). The title should be a heading element (specifically <h1>). Don't be confused by the default styling of it, you can and should overwrite its style as you want. Also notice you should use <h1> element once in a page. The next levels are <h2> <h3>... Also, this should get rid of the accessibility issue in the report.

  Keep it up :)

  Marked as helpful

  0

  ---

  

  Ted•180

  @sharipoff-0-1Posted almost 3 years ago

  @sayednaser Thank you for your feedback, I will definitely take into account to my next solutions :)

  0
• 

  bikeinman•1,080

  @BikeInManPosted almost 3 years ago

  Great Work! I think you nailed it perfectly, apart from @sayednaser's suggestions. Good Luck!

  0

  ---

  

  Ted•180

  @sharipoff-0-1Posted almost 3 years ago

  @BikeInMan Thank you for your feedback. Good luck to you too :)

  0