Skip to content
Submitted about 3 years ago

FlexBox

Eugi•270
@EugiSs
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Single price grid component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello. I recently started learning HTML and CSS. This is my first experience. I would be glad to advise on improvement and any criticism. Thanks :)

Community feedback

  • Dharmik_487•1,740
    @Dharmik48Posted about 3 years ago

    Hey👋,

    I think you should change the unit of width of the .wrapper element to % from vw, it will remove the annoying horizontal scroll bar.

    Apart from that your solution is really good!

    Marked as helpful
    0
    Eugi•270
    @EugiSsPosted about 3 years ago

    @Dharmik48 Thanks for your rating. Remarks are very helpful. I made small changes as you indicated.

    0
  • Alessio Maddaluno•200
    @AlessioMaddalunoPosted about 3 years ago

    Nice job! The only thing I would "fix" is the border-radius of the component (giving that a less radius) and the the shadow color/opacity. Besides that, it's really cool!

    Marked as helpful
    0
    Eugi•270
    @EugiSsPosted about 3 years ago

    @AlessioMaddaluno Thanks a lot for the comment. It helps me. I just made the changes you pointed to.

    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