Four card feature section using CSS Grid

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one. Layout in desktop is much wider, maybe scaling it down would be really great, it is somehow responsive and the mobile layout looks great as well.

  Some suggestions would be:

  • Always have a main element that will wrap the main content of your webpage. This helps users to properly navigate your site. The .container selector could have used main element instead of div.
  • Always have an h1 on a webpage. For this, the text:

  Reliable, efficient delivery
  Powered by Technology
  

  Should have been the h1, you only need 1 h1 per page. So only use 1 h1 to wrap both the text, you can either use max-width on the h1 to make the text wraps on another row, or use a span to wrap the second text and have a display: block on it.

  • When using heading tag, make sure that if you use h4, h1, h2, h3 are present before the h4. Always use heading tag incrementally by 1 level.

  Aside from those, great work again on this one.

  0
• 

  yuenu•505

  @yuenuPosted about 3 years ago

  Hi,

  Nice work, it looks pretty good but the card is too wide on desktop

  I suggest giving them a "maximum width", it will look better

  0