Design comparison
Community feedback
- @BrianMunizSilveiraPosted about 20 hours ago
Feedback for Your QR Code Component Solution
Great work on starting your coding journey and completing Day 1! Your project is well-structured and functional. Here’s some constructive feedback to help you improve:
Strengths
- Well-Structured HTML:
- The HTML is semantic, with clear use of elements like
h4
,p
, anddiv
.
- The HTML is semantic, with clear use of elements like
- Basic CSS Applied Well:
- Effective use of
flexbox
for centering content. - Good choice of colors and spacing.
- Effective use of
- Accessibility Consideration:
- Including
meta viewport
is great for mobile responsiveness.
- Including
Suggestions for Improvement
-
Add Alt Text to the Image:
- To improve accessibility, include an
alt
attribute for the QR code image. Example:<img src="./images/image-qr-code.png" alt="QR Code to access the Frontend Mentor website">
- To improve accessibility, include an
-
Better Centering with CSS:
- Instead of using
margin: 60px 100px;
in.main
, useheight: 100vh
inbody
for precise vertical alignment:body { height: 100vh; padding: 16px; /* Prevents content from being cut off on smaller screens */ box-sizing: border-box; }
- Instead of using
-
Responsive Design Enhancements:
- Add media queries to improve the layout on smaller screens. For example:
@media (max-width: 375px) { img { width: 200px; /* Adjust image size for smaller screens */ } .main { margin: 20px; /* Adjust spacing for smaller screens */ } }
- Add media queries to improve the layout on smaller screens. For example:
-
Font Optimization:
- It seems you’re loading two font families (
Inter
andOutfit
), but onlyOutfit
is used. Remove the unused font to improve performance.
- It seems you’re loading two font families (
Next Steps
- Test your project on various screen sizes to ensure it looks great everywhere.
- Keep experimenting with media queries and accessibility improvements as you progress.
You’re off to a great start—keep up the consistency and learning! 😊
Marked as helpful0@Uyoyo-WPosted about 16 hours ago@BrianMunizSilveira Thank you so much for your feedback! I'll definitely make the necessary corrections
1@Uyoyo-WPosted about 11 hours ago@BrianMunizSilveira I've implemented the corrections. Thank you once again for your feedback! It was really helpful
1 - Well-Structured HTML:
- @Uyoyo-WPosted about 20 hours ago
I'm embarking on a journey of learning how to code all over again and the idea is to write code consistently for the next 30 days by working on projects.
Here's the solution to Day 1. I'd really appreciate feedback on what to do better
1
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