Four card feature section - CSS Grid - Responsive

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • The main element should only be wrapping the card section since that is the main content of your page.

  • The “Reliable, efficient delivery Powered by Technology” is one single heading so the entire thing should be wrapped in a single h1 heading along with a span element.

  • The “icons” serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hide them from assistive technology.

  More Info:📚

  https://www.w3.org/WAI/tutorials/images/

  • Using CSS Grid with Grid-Template-Areas will make things way easier when building the layout; it will give you full control of the layout.

  Here is an example of how it works: EXAMPLE

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  Marked as helpful

  1

  ---

  

  Thiago Barboza Dos Santos•190

  @thgsnakePosted almost 2 years ago

  @vcarames Thank you so much for this tips!

  1
• 

  Favour•2,160

  @Nadine-GreenPosted almost 2 years ago

  HEY THIAGO!

  I noticed that from 451px to 767px, the desktop design shows up instead of the mobile design, just something i thought i could inform you of.

  HOPE I COULD BE OF HELP :)

  HAPPY CODING!

  Marked as helpful

  1

  ---

  

  Thiago Barboza Dos Santos•190

  @thgsnakePosted almost 2 years ago

  @Nadine-Green Thanks so much for letting me know.

  0

  

  Favour•2,160

  @Nadine-GreenPosted almost 2 years ago

  @thgsnake

  Happy I could help

  0