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

Simple responsive page with Grid element

Ewa 30

@em-ewaa

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


I have a question regarding my way of implementing responsiveness - is it a quite a short way of doing this? I still feel unsure with using em/rem/% units and I mostly use my intuition. I'm wondering if there are better ways to approach this.

Community feedback

darryncodes 6,430

@darryncodes

Posted

Hi Ewa,

Nice solution, pretty much spot on!

Some interesting info for you:

A couple of tweaks i'd make:

    background: no-repeat url(/Challenge-1/images/pattern-background-desktop.svg) var(--pale-blue); // I've removed center here
    background-size: 100% auto; // I've added this to match the design and to stretch the background across the whole view port
    margin: 0;
    display: flex; // Added Flex and min-height to responsively center your design, you should remove the margin property on your .bubble class too
    justify-content: center;
    align-items: center;
    min-height: 100vh;

Marked as helpful

1

Ewa 30

@em-ewaa

Posted

@darryncodes I totally forgot about viewport units! I always struggle to make everything centered and responsive. It didn't occur to me to use flexbox on body element but now it seems very clear.

Thank you for your feedback! I will definitely check the links out 🙂

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