Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Component with background animation using CSS keyframes

@BananaTechs

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Wondering if it's a little oversized. Any feedback is appreciated. Thanks!

Community feedback

@MojtabaMosavi

Posted

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 helpful

1

@BananaTechs

Posted

@MojtabaMosavi Thanks Mojtaba! I just looked up shorthand and it looks super useful! ^_^

0

Please log in to post a comment

Log in with GitHub
Discord logo

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