Design comparison
Solution retrospective
Hi fellow coders,
This is my solution to the QR component challenge:) Looking forward to your feedback!
Community feedback
- @Julia-FidisonPosted over 2 years ago
Hello, Great job. I would suggest you to import your google font like @import url('https://fonts.googleapis.com/css2?family=Outfit:wght@100&display=swap'); in your stylesheet as you already have a separate style.css.
And change the color to h1 by Grayish blue: hsl(220, 15%, 55%).
I think you should make you're site responsive by using media query as: 320px — 480px: Mobile devices 481px — 768px: iPads, Tablets 769px — 1024px: Small screens, laptops 1025px — 1200px: Desktops, large screens
You can learn more about that here
Marked as helpful0@sadykovaPosted over 2 years agoHi @Julia-Fidison! Thank you so much for your feedback. I truly appreciate your help. I will make changes to text colors and add media queries. Thank you again!
0 - @JohndiddlesPosted over 2 years ago
Really good job Almira!
just a couple of things I want to point out.
- Typography: I think you can pay some more attention to the text font sizes.
- you can consider specifying a particular width for your texts so that they are consistent across all widths. Personally, I use
ch
as the unit when specifying widths for my texts. You can check out other units also. - for this particular challenge, you need to have a uniform padding across all axis. I see you used 15px for top and bottom and 5px for left and right but you can keep it at 15px on both axis.
- You can also consider using a border-radius twice the border radius of the QR code.
This are just my opinions, definitely not the only way to go about it and maybe not even the best but I think this tips could help improve your solution. overall, nice work you've done. Cheers 🍻 🥂
Marked as helpful0@sadykovaPosted over 2 years ago@Johndiddles Thank you so much for taking time to take a look at my code. I greatly appreciate it. I will make your suggested changes and will see where it takes me :) Thank you again!
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