Skip to content
Submitted almost 3 years ago

Order summary component

Martin de Boer•50
@martindb1988
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 community,

I would like some feedback for the following problems i'm dealing with.

  • SVG background is not showing
  • placement of elements within the .plan class.
  • Improvements regarding the code.

I know this is far from done, but i really could use some community feedback.

Community feedback

  • Robert Rynard•560
    @Robert-RynardPosted almost 3 years ago

    You're background is not showing up because you are not using a relative link in your url. It should be ../images/ I would also recommend putting this on your body so it covers the whole page and you will need to apply some more background properties to make it look correct. To position the items in the .plan class you may want to consider turning it to display flex and positioning them that way You do not need to put .container before all of your classes as they should work without that.

    Hope this helps and good luck with the rest!

    Marked as helpful
    1
    Martin de Boer•50
    @martindb1988Posted almost 3 years ago

    @Robert-Rynard Thank you for your feedback. I will look into this :)

    0

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