Design comparison
Solution retrospective
This was really fun to do ! Please don't hesitate to comment on how I could improve !
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @Fab29012!
your project looks great! I see that you added your special touch to it!
I have a suggestion for you to improve it:
Since all the 3 card share these properties:
border: 1px solid transparent; text-align: center; height: 400px; width: 200px; border-radius: 5px;
You could've created new a class with them and add this new class to each card. This way you wouldn't have to repeat that 3 times. The elements can have more than one class.
So, you can do that:
.new-class { border: 1px solid transparent; text-align: center; height: 400px; width: 200px; border-radius: 5px; }
And then add the class to each card. That's it!
I hope it helps!
Marked as helpful1@Fab29012Posted 11 months agoHey @danielmrz-dev !
Thank you so much for your feedback ! I'll definitely try to remember to implement it into my future works ! That would help clean up my work and allow me to keep track of my styles ! Thank you !
1
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