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 (Pure CSS)

Muhammad Adamβ€’ 210

@Muhammad-adam778

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


Positive feedback is very important for me.

Community feedback

Travolgi πŸ•β€’ 31,420

@denielden

Posted

Hello Muhammad, great work! 😁

Some little tips to improve your code:

  • centering a element with absolute positioning is now deprecated, it uses modern css like flexbox or grid
  • use flexbox or grid to the body to center the card. Read here -> best flex guide
  • after, add min-height: 100vh to body because Flexbox aligns child items to the size of the parent container

Keep learning how to code with your amazing solutions to challenges.

Hope this help! Happy coding!

Marked as helpful

1
Adrianoβ€’ 34,090

@AdrianoEscarabote

Posted

Hi Muhammad Adam, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

Consider using rem for font size .If your web content font sizes are set in absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. Relative units β€œstretch” according to the screen size and/or user’s preferred font size, and work on a large range of devices.

if you want to continue coding with px, you can download a very useful extension in vscode, it converts px to rem! link -> px to rem

To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5

<h1>Improve your front-end skills by building projects</h1>

The rest is great!

I hope it helps... πŸ‘

Marked as helpful

1

Muhammad Adamβ€’ 210

@Muhammad-adam778

Posted

@AdrianoEscarabote Thank you Adriano, i`m very glad for receiving your feedback. About accessibility: Thank you, the problem was solved. About Font Size: l want to clarify one point : There is a potential problem of using only relative units for font sizes is that user-resized fonts may break assumptions your layout makes. For example, this might lead to text getting clipped or running too long. If you use absolute units, you don't have to worry about unexpected font sizes from breaking your layout.

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