Skip to content
Submitted almost 3 years ago

Single Price Grid with Flexbox

DNel•200
@TechEdDan2
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

I welcome feedback about my code... I wonder what best practices, workflow, or ways to approach the solution would be better than they way I did it. Thanks!

Community feedback

  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted almost 3 years ago

    Hi, DNel! 👋

    Nice job on this challenge! 👏 Your solution looks pretty good and is responsive! 😀

    I'd like to suggest,

    • Vertically centering the card component in the viewport by adding min-height: 100vh to the body and then perhaps turning the body into a flex container to easily center the card component inside it.
    • Using a link or button element for the sign-up button instead of a regular div element. Doing so will make your HTML more semantic and help keyboard users to navigate to and interact with that element.

    Hope those one or two tips are helpful. 🙂

    Keep coding (and happy coding, too)! 😁

    Marked as helpful
    0
    DNel•200
    @TechEdDan2Posted almost 3 years ago

    @ApplePieGiraffe Thank you for the feedback and tips!

    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