Design comparison
Solution retrospective
Attention to small details, adding and altering shadows based on screen size, and pixel-perfect matching.
What challenges did you encounter, and how did you overcome them?The challenge I encountered was calculating an adaptive border-radius. I overcame it using calc()
What specific areas of your project would you like help with?none
Community feedback
- @Andre-DMPosted 7 months ago
Hi! This is my first review, but I think I have a couple of tips that may be useful in the future:
-
It is generally a good idea to use rem units for
font size
and px for other properties such aswidth
,height
,padding
andmargin
. This is because rem units are based on the root elementhtml
of the document and are therefore more flexible and responsive than px units. -
To avoid repetition in the code, properties such as
font-family
(in this case, a single font for the entire project) can be placed in thebody
selector, as well default values such asfont-style: normal
can be avoided, unless they are used to override other previously declared font-styles.
I hope this couple tips were helpful to you, otherwise your solution seems great!
Keep up the good work!
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