Design comparison
Community feedback
- @MelvinAguilarPosted about 2 years ago
Hi @msabdalaal ๐, good job completing this challenge, and welcome to the Frontend Mentor Community! ๐
Here are some suggestions you might consider:
- Try to use semantic tags in your code. Click here for more information.:
With semantic tags:
<body> <main class="card"> . . . </main> <body>
- You can use relative file paths in your images, this can save future errors in your images:
<img src="./images/image-qr-code.png" alt="qr code">
- Instead of using pixels in font size, use relative units of measure like
rem
orem
. The font size in absolute length units (px) does not allow any user with limited vision to change the text size in some browsers. Reference.
I hope those tips will help you.
Good job, and happy coding!
2@msabdalaalPosted about 2 years agoHi @MelvinAguilar, thank you so much for the feedback, actually this was my first time trying Frontend Mentor, and I'm so happy I got feedback almost in no time, I will consider your notes they're really helpful, and thanks again โค๏ธ๐
edit: I did some changes to the design, I tried changing all the units to rem even the height of the card and the border radius, and I considered all the notes you've told me ๐
0
Please log in to post a comment
Log in with GitHubJoin 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