Design comparison
Solution retrospective
I would like to know how to write cleaner code in CSS. Your feedbacks are much appreciated =).
Community feedback
- @NehalSahu8055Posted over 1 year ago
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
➨ Replace
background: url(Projects/image-qr-code.png);
withbackground: url(../Projects/image-qr-code.png);
to fix your image issue.
➨ Remove
margins
andpaddings
from.main
as this will not dynamically center the card.- You are already using flex just add
min-height:100vh
to your .page and see the effect.
➨ Use
Semantics
for the proper design of your code.<body> <header> <nav>...</nav> </header> <main>...</main> <footer>...</footer> </body>
-
For
non-decorative images
give meaningful and descriptive alt likealt= "QR code to Frontend Mentor website"
. -
Use
responsive units(rem, em, %)
from next project. Explore respective use cases on google.
link.
I hope you find this helpful.
Happy coding😄
Marked as helpful1@nina1234567896Posted over 1 year ago@NehalSahu8055 this is very helpful indeed! thankyou so much. I am rather confused by relative paths. Is there any way i can learn more about writing paths?
1 - @NehalSahu8055Posted over 1 year ago
Also
- After making the necessary changes generate a new screenshot
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