Design comparison
SolutionDesign
Community feedback
- @OlaOluwalekanPosted over 2 years ago
Nice try!
However, you added to much padding into your qr-code-container class gene, there's little space for it to occupy. Also the border radius you added is somehow awkward. You could have used something like 10px for the qr-code-container and 5px for the image.
Also, you didn't stick strictly to the styles in the style-guide.md
Marked as helpful1 - @satyammjhaPosted over 2 years ago
Great attempt @Arearda. I have some suggestions for you:-
- use the
<br>
tag after the word front-end so that other skills will move to the next line. - use the
<br>
tag after the word frontend and to in the paragraph. - Use
color: rgb(135, 135, 135);
for the paragraph. - Your design is not responsive. Use media queries to make it responsive according to the screen resolutions.
- For fonts and colors used in the projects please read the style guide of the project before coding.
Hope these tips will help you. Happy Coding 🤗✌🏻
0 - use the
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