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 component

Raihan-21 210

@Raihan-21

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


any feedbacks appreciated thanks!

Community feedback

P
Chamu 12,970

@ChamuMutezva

Posted

Welldone Kucabara21. Here are some few suggestions on things that i noted:

  • For some medium devices , there is a large whitespace at the top which is caused by the justify-content: center on the body. check at screen width of about 935.
  • Heading elements are supposed to be in order. The first level heading is supposed to be h1 followed by h2 and so on. This is important for accessibility among other things. For further reading see link [https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements] If you find this helpful please upvote me Happy coding
2

Raihan-21 210

@Raihan-21

Posted

@ChamuMutezva oh wow thanks for the heading info so i should've just edit the font size? i cant find any whitespace at 935 width screen? I already upvoted ur commment thanks

0
P
Chamu 12,970

@ChamuMutezva

Posted

@Kucabara21. Yes - use css to style the font to any required size. Let me see if I can post an image of the screenshot.

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