Responsive Solution with HTML & CSS

Community feedback

• 

  P

  littlesamm96•50

  @littlesamm96Posted 2 months ago

  Readable, well-structured code although I would use more descriptive text for alt attributes of images for screen readers to implement accessibility.

  Marked as helpful

  0

  ---

  

  Blessing Akanimoh James•230

  @JahsbabyPosted 2 months ago

  @littlesamm96 noted boss, I'll implement the changes

  0
• 

  repro123•600

  @repro123Posted 2 months ago

  Nice work. Some corrections.

  • It is a wrong practice to limit the body to a specific height. Use min-height: 100dvh instead. On a larger project, that height you used on the body will cause bugs.

  • This challenge does not need a media query... However subsequently, if you have to use a media query, they must be in responsive units like rem.

  • Use responsive units for font sizes too. Rem is fine. /* 1 rem = 16px*/

  • finally, also try and use a Modern CSS Reset at the top of your stylesheets. You can check out Andy Bell's Modern CSS reset or the one by Josh Comeau.

  Great job, once more

  Marked as helpful

  0

  ---

  

  Blessing Akanimoh James•230

  @JahsbabyPosted 2 months ago

  @repro123 thank you so much boss

  I'll make the needed the correction

  0
• 

  Muhammad Awais•1,100

  @muhammadawaislaalPosted about 1 month ago

  good its amazing

  0