Responsive QR code component using CSS Flex

Community feedback

• 

  Fer•3,970

  @fernandolapazPosted over 1 year ago

  Hi 👋🏻, some of this may interest you:

  • The image is not decorative and therefore the alt attribute should have at least a short description, like 'QR code to visit Frontend Mentor'.

  • It is better to use min-height: 100vh for the body as using height causes the page to be cut off in viewports with small height (such as mobile landscape orientation).

  • Regarding what you mention about reducing unnecessary code, the only thing I see is that you could remove font-weight: bold from the <h2>.

  I hope it’s useful : )

  Regards,

  Marked as helpful

  0

  ---

  

  Mahmoud_Amin•540

  @Mahmoudamin11Posted over 1 year ago

  @fernandolapaz this the most useful comment i have ever recieved thank you : )

  1