H.Lite• 40
@Henock-Lite
Posted
The code is well structured and more readable after simplification. Removing redundancies and consolidating font rules in the body improves clarity.
Marked as helpful
0
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
I am proud of myself because I was able to achieve this challenge. There is one thing I will do differently next time, it is to practice each topic as many times as possible, what I mean by this, it is not just studying flexbox, grip or bootstrap, because today I realize that I found it difficult to remember any tips about flexbox, mainly because the last topic I have studied is bootstrap.
What challenges did you encounter, and how did you overcome them?
As I mentioned before, I used flexbox and had forgotten some important tips about it. The quickest way to get through this on my own was to read my notebook again and do some practice.
What specific areas of your project would you like help with?
simplify my code
Community feedback
H.Lite• 40
@Henock-Lite
Posted
The body is already centered, so the .container can be simplified to a simple container class without affecting the layout
The font styles (font-family) are removed from the h2 and p elements since they are already defined in the body.
Marked as helpful
0
voespinal-code• 50
@voespinal-code
Posted
Thank you, I appreciate your comment.@Henock-Lite
0
homosapiensapien• 20
@homosapiensapien
Posted
Only thing is that the QR code is far larger than the original design.
0
yesinite• 30
@yesinite
Posted
Well structured, good semantic code.
0
Please log in to post a comment
Log in with GitHub
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