Skip to content
Submitted about 3 years ago

Mobile-first, flexbox, html5, css3

Caroline SENES•65
@CarolineSenes
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

I did not have any particular difficulty for this project. But if you have any comments, let me know :)

Community feedback

  • Fluffy Kas•7,715
    @FluffyKasPosted about 3 years ago

    Hey, your solution looks good! Few things I noticed:

    1. The background image doesn't work in desktop view.

    2. A cursor: pointer would look nice on the button.

    3. You use IDs but actually simple classes would do the job just as well.

    Marked as helpful
    0
    Caroline SENES•65
    @CarolineSenesPosted about 3 years ago

    @FluffyKas

    • The background works up to 1440px which is the maximum resolution of the image. In order not to have it cut, I chose to remove it after 1440px.
    • Thanks for the button cursor! 👍 I fixed it.
    • You're right! 😆 I changed the ID to Class.
    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