Responsive four card feature section using CSS Grid and Media Queries

Community feedback

• 

  Ken•935

  @kenreibmanPosted about 3 years ago

  For small projects like this I believe media queries are fine.

  You are also missing an h1 in your entire project. All HTML projects should start with an h1. In my project, I set the h1 to "Powered by Technology", h2 was "Reliable, efficient delivery" but that was my vision.

  You should never use heading tags depending on their style, size, and font weight. Always focus on having that correct heading hierarchy. You can set style and all that in CSS later.

  Whenever you use a section tag, you should always have heading elements as child elements or it isn't semantically correct. In your case, <section class="cards-section"> lacks heading tags. Instead of using article elements, I would just turn them into divs.

  Marked as helpful

  0