Responsive QR Code Component built with basic CSS properties

Community feedback

• 

  optimusprime202•1,200

  @optimusprime202Posted over 2 years ago

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

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @optimusprime202 Thank you

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 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 👍!

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @Crazimonk Thank you

  1
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted over 2 years ago

  In image accessibility give the image alternative name like👇

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

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @GitHub-dev12345 Thank you

  0
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted over 2 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)

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @GitHub-dev12345 Alright, thank you!

  0
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted over 2 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;

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @GitHub-dev12345 Alright, thank you!

  0
• Account deleted

  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!

  Marked as helpful

  1

  ---

  

  Chiamaka Mary-Anne Ugwu•120

  @chiamaka-ugwuPosted over 2 years ago

  @Old1337 Thank you so much!

  0