Single price grid component using CSS, Grid

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey @ClaireKarsenti, some suggestions to improve you code:

  • This component should be built using CSS Grid as the name of the challenge implies.

  • The “30-day, hassle-free money back guarantee” is not a heading. It should instead be wrapped in a Paragraph Element.

  • Remove all the commented code out from you HTML code, all it does it clutter up your code.

  • You created your button using both the Button* and Anchor Element. This is the wrong way of creating a button since only one element can be use. For this challenge you will go with the Anchor Element.

  • The button's :hover color is incorrect. You want to check you "style-guide" to get the correct colors.

  • The “Why Us” list should be created using an Unordered List Element along with the List Items Element.

  • Implement a Mobile First approach 📱 > 🖥

  With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.

  Happy Coding! 👻🎃

  Marked as helpful

  0

  ---

  

  Claire Karsenti•130

  @ClaireKarsentiPosted about 2 years ago

  Hey @vcarames !

  Thank you for reviewing my code :-)

  I updated my code followed your suggestions :

  • I have rebuild the project with Grid instead of Flexbox.
  • I implemented it with the mobile first approach.
  • I adjusted the hover state.
  • I updated the button element.
  • I removed my commented code even if I thought It might be helpful for the frontend mentor community who may have a look on my repo.

  0