Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

QR code component using Vanilla CSS and Flexbox

Mo 840

@MohamedAridah

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello Coders!, How do you find my code? if anything need fix ,please let me know..

Have a good day everyone^^

Community feedback

Ciceron 940

@MarcusTuliusCiceron

Posted

Hi, congrats on completing this challenge,

Your code is really clean, one little remark (its really not a big deal), rem unit are not always adapted to width and height and will make design less responsive, it is better to stick to %, vh, vw, vmin or other relatives unit. rem is usually better for font size and line height.

Hope this will help, and again this isn't really a big deal.

Happy coding :D

Marked as helpful

0

Mo 840

@MohamedAridah

Posted

@MarcusTuliusCiceron i really appreciate your notes and will take it in consideration in the other projects.👍😍 So if i got you right, i should use relative units instead of rem especially with width and height, is that right?

Have a nice day^^

1
Ciceron 940

@MarcusTuliusCiceron

Posted

@MohamedAridah yes, rem is also a "relative" unit but it's relative to user preferences. Size of other element needs to be relative to viewport size to ease the responsiveness (you still need some breakpoint to modify the display for different device)

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord