QR Code using bootstrap 5 and custom CSS

Community feedback

• 

  faizan•2,420

  @afaiz-spacePosted over 2 years ago

  Hey @PandaMajor,

  • use flex or grid property to the object center in the body element. no need gives position property use.

  Marked as helpful

  1
• 

  Afan Khan•70

  @justmrkhanPosted over 2 years ago

  Hello,

  1. To understand your question, it is acceptable if you use bootstrap. It saves time on big projects. However, it is not important in small component challenges like these.
  2. Try not to keep links or alt texts empty. Also, you haven't added alt text at all. I suggest adding alt text for screen readers.
  3. For accessibility purposes, you must add the role tag in main divs and sections for screen readers.

  Other than that, I think the other feedbacks cover more topics.

  Marked as helpful

  0
• 

  RPotata•40

  @RosalkoPosted over 2 years ago

  I think there is no harm in using bootstrap components, it just might be a little overkill for such small component, and practising creating such card with pure css might be a good exercise :).

  The text doesn't have to always match 100%, but for me it was simply trial and error.

  Also, don't forget to use the alt attribute for imgs, as mentioned in the html report ;)

  Marked as helpful

  0