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 with flex-box

Mulugeta 360

@MrDevM

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


done with flex-box am waiting feedback

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Your site and code look great. Good job!!!

Some things I would change with regards to your website:

  • you didn't use a <h1> tag on anything and skipped straight to <h2>. You should change your headers so that there is at least one <h1> tag as headers are meant to be used in order with out any skips. You can always customize the font-size in css with the font-size property.
  • you should add a hover effect to your button using the :hover feature so your site has more interactivity
  • for the third box ("Why Us"), you should use an unordered list to list everything out because right now when the screen size changes there are some instances where the text is just one big paragraph. An unordered list is something like this:
<ul>
   <li>Item 1</li>
   <li>Item 2</li>
   <li>Item 3</li>
</ul>

Hope you find this feedback useful 👍

Marked as helpful

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