Skip to content
Submitted about 3 years ago

css Grid, html, flexbox

accessibility, angular, backbone, bootstrap, express
Eric Wisdom Agu•20
@Ericwiz
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

please check out my solution and suggest areas of improvement.

Code

Please log in to post a comment

Log in

Community feedback

  • Marius•190
    @MariusHor

    Great job so far! 🙂

    for the background you could add these lines: background-color: hsl(225, 100%, 94%) background-size: contain background-repeat: no-repeat.

    I would also suggest that you dont set fixed heights and widths on elements. Use padding to create space within an element and margins to create space between elements and not top left or position: relative / position: absolute. Google "spacing in CSS" for a better understanding.

    Keep going, you are getting there! 🙂

  • clive•190
    @clivepato93

    Hi there for your background-repeat use no repeat

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