Skip to content
Submitted almost 2 years ago

order-summary-component-main using native-css

Basit Korai•350
@basitkorai
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 really enjoyed coding this challenge.

Things I used for the first time: CSS Custom Properties, Transitions

Things I'm unsure of: Accessibility, Using lots of divs

Feel Free to give me any tips and suggestions regarding this challenge.

Community feedback

  • Adriano•38,160
    @AdrianoEscarabotePosted almost 2 years ago

    Hi Basit Korai, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

    To prevent the background image from breaking at higher resolutions, we can prevent this in two different ways:

    1. Add a background-repeat: repeat-x;, the image will repeat on the horizontal axis, preventing it from breaking.
    2. Add a background-size: 100% 50vmin;, the 50vmin will set its height as the page target, and 100% will make it stretch on the horizontal axis.

    Feel free to choose one of the two!

    Add a padding to the body, so that the content in lower resolutions doesn't hit the edge of the screen!

    The rest is great!

    I hope it helps... 👍

    Marked as helpful
    1
    Basit Korai•350
    @basitkoraiPosted almost 2 years ago

    @AdrianoEscarabote Thank You for your feedback! Love you brother. I'll fix it.

    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