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

Single price grid using HTML and CSS (Flexbox)

Jose Angel Reyβ€’ 395

@Jose-Angel-Rey

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


Hello everyone!! This is my second Frontend Mentor challenge and I would like to receive your feedback regarding:

β€’ HTML structure πŸ—

β€’ Responsive design πŸ–₯πŸ’»πŸ“³

β€’ Application of BEM

and if you have any recommendations about something that i need to improve it would be great πŸ‘πŸΌ

Community feedback

Mijail Hernandezβ€’ 950

@MishaHernandez

Posted

Hi Jose!

  • You are using flexbox in the body to center the main container horizontally and vertically but the container has margins assigned (margin: 10% 0px 5% 0px) so a vertical scroll is generated in the desktop view. An alternative is to remove these margins and assign the body a height: 100vh. Greetings and continue with the challenges.
3

Jose Angel Reyβ€’ 395

@Jose-Angel-Rey

Posted

@MishaHernandez Thank you so much for your feedback πŸ™πŸΌ . I already fixed the errors with the margins. I hope you can see it again πŸ‘¨πŸΌβ€πŸ’»

0
Mijail Hernandezβ€’ 950

@MishaHernandez

Posted

@Jose-Angel-Rey Thats very good! It is totally centered πŸ‘

0

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