Html and Css

Community feedback

• 

  Joseph•770

  @Jos02378Posted about 3 years ago

  Hey @ChitThetAung, great job on this solution!

  Some suggestions for you:

  1. You can use the main element to wrap your code in the body to clear some accessibility issues in the report.
  2. You don't need to add alt text for the images because they are for decorative purposes.
  3. You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
  4. You can try to use relative units like em in the future. Here is a link to an article that explains when to use which unit see the article.

  I hope this helps, good luck!

  Marked as helpful

  0

  ---

  

  Chit_Thet_Aung•195

  @ChitThetAungPosted about 3 years ago

  @Jos02378

  Thanks for suggest me. I will watch your suggested video.

  0