Responsive 3-card layout

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Gonca! 👋

  Congrats on finishing another challenge! 🎉 Your solution looks very good and also responds well. Here's my few tips:

  • You don't have to wrap every element into div. Add .card-header to h2 element instead. It also apply to .card-paragraph and .card-button divs. If it comes to image you can keep it in additional wrapper.
  • Change the alt attributes for the .card-icon images, as they don't add any extra context for screen reader users. Since your images are decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  Gonca•100

  @goncanegisPosted over 3 years ago

  @tediko you’re absolutely right, thank you so much for your detailed and insightful comment!

  0
• 

  Shiryumee•210

  @Shiryu790Posted over 3 years ago

  I saw it on my Phone and my computer. It's responsive and the layout corresponds to the requested design

  1

  ---

  

  Gonca•100

  @goncanegisPosted over 3 years ago

  @Shiryu790 awesome, thank you for taking the time to comment!

  1