Responsive QR code done with flex and palin CSS - Updated 22/11

Solution retrospective
Hi, /* This is an updated version after taking notes from _nehal💎, thank you for the helpul review:)*/ For this exercise I wanted to practice plain CSS and Flexbox. Hope you like it and if you have constructive criticism it will be very much appreciated. Best Regards, Gabriel.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @NehalSahu8055
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
-
Add
min-height:100vh
to your body to dynamically center the card. -
Every site must have one h1 element describing the main content of the page.
-
So, Add a
level-one heading
-
<h1>Improve you frontend skills by building projects</h1>
-
Use
Semantics
for the proper design of your code.
<body> <main>... main content goes here ...</main> <footer>...</footer> wrap up `.attribtution(challenge by....)` div inside your 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.
I hope you find this helpful.
Happy coding😄
Marked as helpful -
Join 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