Used HTML, CSS(flexbox),& no framework/libraries.Excited to learn more

Community feedback

• 

  Patrícia Silva•105

  @patriciarrsPosted 4 months ago

  Greetings, Arunstev! 👋

  Good effort on this challenge! 👍 The layout looks good on a range of screen sizes 🙂

  I want to suggest taking another look at using semantic HTML on your site as it will also improve accessibility. Currently, you are using div tags to wrap other elements, but other tags (for example, main and article) are more suitable.

  If you’d like to learn more about how to use semantic HTML on your site, check out this course from web.dev.

  The solution has differences from the design. The most noticeable is the font family.

  To correctly import the specified font family, you can go to the link you have in your @import in the CSS file and then click "Get font" -> "Get embed code".

  I hope you find this helpful.

  Happy coding!

  Marked as helpful

  0

  ---

  

  Arunstev•30

  @ArunstevPosted 4 months ago

  "Hi! Thanks for the feedback! I'll review my HTML and replace div tags with semantic elements like main and article, etc. I'll also update the font family with the correct embed code. Appreciate your help!

  1