@vanzasetia
Posted
Excellent work on this challenge, Geraldo! Your solution is responsive and looks great! π
In your CSS, I noticed this selector .container .card .card-2
which would be much be as .card-2
. I would strongly recommend keeping the CSS specificity as low and flat as possible. High specificity will make your stylesheet hard to maintain.
Try to use rem
or sometimes em
unit instead of px
. Using px
will not allow the users to control the size of the page based on their needs.
I highly suggest writing the styling using the mobile-first approach. It often leads to shorter and better performance code. Also, mobile users won't be required to process all of the desktop styles.
That's it! I hope this information is useful! π
Marked as helpful
@Geeraldv
Posted
@vanzasetia Your comment is very useful, these points will make me improve a lot as a frontend, I hope more reviews in my challenges. βπ»π¨π»βπ»