Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…
- This part could be simplified into...
<div class="qr-title"> <h1>Improve your front-end skills by building projects</h1> </div> <div class="description"> <p> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </div> Into: <section class="sample name here"> <h1>Improve your front-end skills by building projects</h1> <p> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </section>
- The font size of
h1
can be adjusted intofont-size: 20px;
- Add the
background: var(--Light-gray);
in thebody
rule set
On the contrary:
- The HTML structure is done well. You utilized the semantic tags properly
- The code is clean and neat
- The design's details is well implemented
- Used proper link for the font style and weights
- The CSS structure is well organized.
Above all, The design looks good. Keep it up!
Marked as helpful1@saifion33Posted almost 3 years ago@RioCantre Thanks Rio your feedback is helpful for 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