Skip to content
Submitted over 2 years ago

Single price grid component using flexbox 1st attempt

Hannah Oā€¢60
@hannahro15
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 don't think this is too bad but I had a few problems with finding the correct green colours as there was only one type of green in the style guide and the fonts I don't think look quite right. Any feedback other than this would be useful too. Thanks.

Community feedback

  • Shashree Samuelā€¢9,260
    @shashreesamuelPosted over 2 years ago

    Hey good job completing this challenge

    Keep up the good work

    Your solution looks great however I think the following should be considered

    • your card needs some padding around the content

    • the card needs a subtle box-shadow using the box-shadow property

    • Your font-family does not match the one specified in the style-guide.md file. I recommend rechecking the file to be sure

    In terms of accessibility issues simply wrap all your content between main tags

    I hope this helps

    Cheers Happy coding šŸ‘

    Marked as helpful
    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