Design comparison
Solution retrospective
It was a fun project to complete. I think next time, I would try to practice using a framework possibly.
What challenges did you encounter, and how did you overcome them?I didn't encounter too many challenges on this assignment.
What specific areas of your project would you like help with?Getting the sizes (like the width of the card) correct.
Community feedback
- @vanzasetiaPosted almost 2 years ago
Hi, DNel!
Congratulations on completing this challenge!
Nice alternative text for the QR code! It is rare to see people that can write good alternative text. Amazing! 👏
I recommend removing all commented code. If other developers or yourself in the future work with the same code base, they can get confused about whether the commented code should be used or deleted.
I recommend making the
body
as a flex container to place the card in the center of the page. Then, there is no need for a wrapper to havemargin-top: 15%
. Be aware that using percentage unit formargin
andpadding
can lead to unexpected results on some screen sizes. It is because the value (computed value) is relative. So, it is better to use a static unit (rem
) to make sure it is consistent across screen sizes.I suggest using a CSS reset whenever you start a new project. This can help you set the styling foundation easily.
Read more — A Modern CSS Reset
I hope this helps. Happy coding!
Marked as helpful1@TechEdDan2Posted almost 2 years ago@vanzasetia Thank you for all of your helpful notes and suggestions.
0@vanzasetiaPosted almost 2 years ago@TechEdDan2 Happy to hear that was helpful! 😊
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