Skip to content
Submitted almost 3 years ago

Order summary component

Olabisi•70
@olabisi09
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

Community feedback

  • Anosha Ahmed•9,300
    @anoshaahmedPosted almost 3 years ago

    Hey good job on this challenge! Your report above shows a few issues. You can fix them by

    • wrap everything in your body in <main> ... OR use semantic tags! .... you can also give role="" to the direct children of your <body> but that's a little frowned upon... Read more here
    • have at least one <h1> in your code
    • add alt="" to your <img> tags

    Hope this helps :)

    Marked as helpful
    0
  • Rio Cantre•9,690
    @RioCantrePosted almost 3 years ago

    Hello there! Good job making it this far. Your design is great and looking at your solution I think you should considered the following as well...

    • Inside the rule set body, html adjust the height into 50% that is height:50%; and add the background-color as well.
    • Add a hover state with the .button1 property
    • Add cursor: pointer for both .button1 and .button2
    • On every image used , include an alt to specify the image
    • Wrap the container with main tag and attribution with footer tag

    Overall you did well, and Keep going!

    Marked as helpful
    0
    Olabisi•70
    @olabisi09Posted almost 3 years ago

    @RioCantre thank you so much!

    0
    Rio Cantre•9,690
    @RioCantrePosted almost 3 years ago

    @olabisi09 No worries. I would also appreciate if you mark it as helpful. Cheers!

    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