qr code component

Community feedback

• Account deleted

  Very nice work ✨, Congratulations

  here are some suggestions:

  • you can center with modern ways, like using grid or flexbox rather than using absolute, left, top, transform and translate
  • you needn't to make div inside another div just to guve it another class name, you can use on div and assign css properties to it (you should remove the div that has the content class, and you have to assign content properties to the container class
  • you should use HTML5 semantic tags, use <main> instead of the first <div>, and you can use <section> for the other divs

  Good luck in the next 🔥, keep going on

  Marked as helpful

  0

  ---

  

  mohamedbntaha•70

  @mohamedbntahaPosted over 1 year ago

  @MasterMousa Thanks for the advice, especially the first tip. I will use it in future projects❤❤

  1

  Account deleted

  you're welcome @mohamedbntaha

  good luck brother <3

  0