Skip to content
Submitted over 3 years ago

Mobile first, flexbox

Mikko•15
@mikkoo99
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

Hi there! I'd really loved to get feedback from you, about my code. It's my first submitted project on frontendmentor.io Could you check my code in context of flexbox? I know on some mobile devices there is issue with displaying content.

Community feedback

  • bunee•2,020
    @buneeIsSloPosted over 3 years ago

    Hi! @mikkoo99, Congratulations on completing your first challenge, You've done great! The page responds very well and you're using flexboxes the right way. The only thing I'd suggest here is that, You can add a max-width to your container... That way the container won't stretch too far. Hope this helps :)

    Marked as helpful
    2
    Mikko•15
    @mikkoo99Posted over 3 years ago

    @buneeIsSlo thanks a lot, I will add a max-width😌

    1
  • Yashasvi Singh•890
    @aUnicornDevPosted over 3 years ago

    You can center the per month text after 29$ using flex. And don't use <br> tag for the next line. Use a new <p> tag instead.

    Marked as helpful
    1
    Mikko•15
    @mikkoo99Posted over 3 years ago

    @aUnicornDev thanks for comment! Why should I not use br tag?

    0
  • Andrei•110
    @retiens123Posted over 3 years ago

    Great job! I don't have any problem to display on my mobile. Keep up!

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