Single price grid component using HTML, SASS, JS and BEM.

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted almost 3 years ago

  Hey, really nice work on this one. The desktop layout looks really great I think and the site is responsive which is really nice. The mobile state looks great as well.

  Here are some suggestions on the site:

  • body tag has a font-size: 400 which is an invalid value, check that one out.
  • Removing the position: absolute from the footer would be really nice so that it won't get in the screen's way. If you try to inspect the site from dev tools at the bottom, the footer is stuck in front of the main-content.
  • Remember that for every page of a site, a single h1 is needed. For this one, you could replace the first h2 to an h1 so that it would be present in the dom. Just remember to change the second text below it to a higher heading tag level or just use p tag on it.

  Those only since the site looks really nice to be honest. Again, awesome work on this one.

  Marked as helpful

  0

  ---

  

  Esteban Canaza•200

  @stebancPosted almost 3 years ago

  @pikapikamart Thank you for your comment and suggestions! This helps me write better code.

  1