Design comparison
Solution retrospective
This is my first challenge. I've centred everything on the screen with flexbox. I haven't done any semantic in this one, but definetly start doing it with the future projects. Funny thing about this first one is that I thought that I have to recreate QR Code by using CSS and HTML only.
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hi there π. Good job on completing the challenge ! I have some feedback for you if you want to improve your code.
HTML:
- Use the
<main>
tag to wrap all the main content of the page instead of the<div>
tag. With this semantic element you can improve the accessibility of your page.
CSS:
- In my opinion, the div with the
container
class is unnecessary, because you can place all its styles directly in the body element and have the same result.
- Setting the font-size to 62.5% can attract compatibility issues with third-party libraries or plugins. You can read more about this with this two lectures:
Credit to grace-snow and vanzasetia for pointing this out.
- You should use the
border-box: box-sizing
property to make thewidth
andheight
properties include the padding and border of the element. This will make it easier to calculate the size of an element. You can read more about this here.
- You should use a CSS reset to remove the default browser styles and make your page look the same in all browsers.
Popular CSS resets:
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding and Happy New Year! πππ
Marked as helpful1@sztosiurPosted almost 2 years ago@MelvinAguilar Thanks a lot for your feedback. It is really valuable.
0 - Use the
- @manishdevelopsPosted almost 2 years ago
congrats on completion of your very first project. Your solution is quite good. However there is some responsiveness problems in small devices. Add these codes to achieve full responsiveness.
.wrapper { width: min(239px, 100%); } .qr { width:100%; }
and remove max-width from the .text class
Marked as helpful0
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