Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Mobile first, flexbox

Mikko 15

@mikkoo99

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@buneeIsSlo

Posted

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

@mikkoo99

Posted

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

1

@aUnicornDev

Posted

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

@mikkoo99

Posted

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

0
Andrei 110

@retiens123

Posted

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
Discord logo

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