Skip to content
Submitted about 1 year ago

Annual plan using css grid

Cyuzuzo Bienvenu•220
@bienvenudev
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello guys! I just finished the Order summary card challenge and I would like your feedback on this: Alt text: I tried my best to make them meaningful but I know I can improve. Naming classes: I tried using the BEM naming but still going back to my codes and finding a specific class is giving me a headache. Annual plan: I used grid on the annual plan but the spacing was far from what I wanted so I used padding inside. Any suggestions on this and anything else are welcome!

THANK YOU!

Community feedback

  • Celestino Trosso•200
    @iamcelestinoPosted about 1 year ago

    @media (min-width: 640px) { body { background: url(images/pattern-background-desktop.svg); background-repeat: no-repeat; background-color: var(--clr-300); } }

    it´s advisable to define min or max-widths in ems, and another Thing is that the background image size should be 100%, in Mobile and desktop.

    1

Please log in to post a comment

Log in with GitHub

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