Submitted almost 3 years ago
Component with background animation using CSS keyframes
@BananaTechs
Design comparison
SolutionDesign
Solution retrospective
Wondering if it's a little oversized. Any feedback is appreciated. Thanks!
Community feedback
- @MojtabaMosaviPosted almost 3 years ago
1-It's a little bit oversized for the amount content it's containing, you could reduce the max-width to something like 400px.
2- Using css shorthand properties is great way to reduce the volym of you styles, for instance using background you merge background-image,background-size and ect.
Keep coding :=)
Marked as helpful1@BananaTechsPosted almost 3 years ago@MojtabaMosavi Thanks Mojtaba! I just looked up shorthand and it looks super useful! ^_^
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