Responsive QR Code Component built with basic CSS properties

Community feedback

• 

  optimusprime202•1,160

  @optimusprime202

  Posted over 3 years ago

  Hey @chiamaka-ugwu, Keep up the good work.

  11

  Marked as helpful
• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 years ago

  Hay ! Chiamaka Mary-Anne Ugwu Good Job on challenge

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body <main class="container"></main>

  -> Always use the "alt attribute" and write what img is , and if the img is for decorative then leave it empty but always add it with img tag.

  -> Learn more on accessibility issues

  If this comment helps you then pls mark it as helpful!

  Have a good day and keep coding 👍!

  11

  Marked as helpful
• 

  Dev Rathore•2,540

  @GitHub-dev12345

  Posted over 3 years ago

  In image accessibility give the image alternative name like👇

  <img src="./image/bill.jpg" alt="bill gates">

  11

  Marked as helpful
• 

  Dev Rathore•2,540

  @GitHub-dev12345

  Posted over 3 years ago

  Reduce accessibility to change this code 😊👇

  <div class="box"> to <main> ( used main tag). <div class="attribution"> to <footer> ( Uses footer tag for footer design)

  11

  Marked as helpful
• 

  Dev Rathore•2,540

  @GitHub-dev12345

  Posted over 3 years ago

  Congratulation Nice Work Keep it up 😉👍

  Used this code want your card in center position :

  in body tag Used this CSS Code: body{ display : flex; justify-content: center; align-item: center; }

  in Card Design CSS Code used this property: align-self: center;

  11

  Marked as helpful
• Account deletedPosted over 3 years ago

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info
  • All Images must have alternate text

  i hope this is helpful and goodluck!

  11

  Marked as helpful