Skip to content
Submitted about 3 years ago

HTML Semantic markup, mobile first styling.

Camille•130
@fyrfli
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Fixed the grid spacing for the price plan line for the desktop version.

Code

Please log in to post a comment

Log in with GitHub

Community feedback

  • Shashree Samuel•8,860
    @shashreesamuel

    Hey fyrfli good job completing this challenge. Keep up the good work.

    Your design looks great however in my opinion you might want to consider increasing the value for the margin top to get the card in the exact center of the page.

    In terms of your accessibility issues

    • heading tags must only increase by one level
    • anchor tags <a> cannot be nested in button tags <button> since both tags are interactive.

    Once you correct your accessibility issues your validation errors should disappear.

    I hope this helps

    Cheers Happy coding. 👍

    Marked as helpful
  • Naveen Gumaste•10,420
    @NaveenGumaste

    Hello Camille ! Congo 👏 on completing this challenge

    Just one thing:

    • The right of the "change" is have more padding or margin so reduce it.

    happy Coding😀

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