QR code component using Flexbox

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey Alberta!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should use <main class="comp"> instead of <div class="comp">.
  • Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
  • You should likely use the font recommended in your style.md file
  • You can also reduce the font-size of your header

  This should fix most of your accessibility issues

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  Marked as helpful

  1

  ---

  

  Alberta Ogbo Dadeboe•50

  @Ogbo0810Posted over 2 years ago

  @Samadeen Thanks for the feedback. I actually used the recommended font family but I'm yet to find out why it hasn't reflected yet. Also, thanks for drawing my attention to the headings and semantics.

  0
• 

  anas chaouki•110

  @anaschaoukiPosted over 2 years ago

  Looks nice, though your font should be the one from the style guide other than that good job!

  0

  ---

  

  Alberta Ogbo Dadeboe•50

  @Ogbo0810Posted over 2 years ago

  @anaschaouki Thanks so much for the feedback. I included the font style in my CSS file but it hasn't reflected. I'm yet to review and find out why

  0

  

  anas chaouki•110

  @anaschaoukiPosted over 2 years ago

  @Ogbo0810 Try replacing the imported font with this

  @import url("https://fonts.googleapis.com/css2?family=Outfit:wght@400;700&display=swap");

  Marked as helpful

  0