Skip to content
Submitted over 2 years ago

Used Flexbox and then the rest is just basic html/css.

TheMightyChungus•190
@TheMightyChungus
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

It was completed in a very messy way.

I'd like to know if there was a more efficient use of flexbox I could have used to use less code.

I'm not a fan of using margin the way I did - any comments on that would be appreciated.

Used position: relative a lot. If there is a way to cut down on that, that'd be nice to know too.

Community feedback

  • kelvinhe04•340
    @kelvinhe04Posted over 2 years ago

    You don't have to specify a height on the "id=main" unless you didn't specify width. Try learning flexbox, which will eliminate all those margins you used, without it would be complicated to replicate this challenge.

    1
  • Marcos Porto Rafael•110
    @mrcsportoPosted over 2 years ago

    Try to use GitHub Pages to upload ->> https://docs.github.com/en/pages/quickstart

    1
    TheMightyChungus•190
    @TheMightyChungusPosted over 2 years ago

    @mrcsporto

    Thanks for the suggestion.

    Had the same problem.

    Figured it out though.

    CSS wasn't getting applied because I changed file names moving stuff around.

    The HTML, therefore, wasn't using the correct path to get to the CSS page.

    That fixed it.

    https://themightychungus.github.io/frontendmentor/

    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