Skip to content
Submitted over 3 years ago

order-summary-component

j5•2,290
@jmnyarega
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

Any feedback is much welcomed 🙏

Community feedback

  • Vanza Setia•27,775
    @vanzasetiaPosted over 3 years ago

    👋Hi Josiah!

    I have one feedback on this solution. Annual plan should be a strong tag not a heading. Heading tag is only for titling each section.

    That's it! It's good that you're using rem unit. Hopefully this little tip is helpful for you!

    Marked as helpful
    1
    j5•2,290
    @jmnyaregaPosted over 3 years ago

    @vanzasetia Thanks Vanza, this is useful.

    0
  • Patrick•14,265
    @palgrammingPosted over 3 years ago

    you should change your background-size from initial to background-size: 100vw 40vh;

    Marked as helpful
    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