3 Column Preview Card Component using Tailwind CSS

Community feedback

• 

  Vanza Setia•27,715

  @vanzasetia

  Posted over 3 years ago

  Hi, Deepak! 👋

  Congratulations on finishing this challenge! 👏 Nice work on this challenge! 🙌

  Your solution looks good on both landscape and portrait mobile views and on my desktop view! 👍

  I would recommend writing your own styling for the newbie challenges to strengthen your CSS skill.

  I have some feedback on this solution:

  • <img src="images/icon-suvs.svg" width=64 height=40 alt /> I would recommend wrapping the value of the width and height properties with double quotes and making the alt="" look like that. <img src="images/icon-suvs.svg" width="64" height="40" alt="" />.
  • Every page should only contain one h1 as an identifier of each page. So, change all the current h1 to h2, and like @AndyGlz01 has said earlier, if this is a full website, the h1 would exist somewhere else on the page.
  • Create a custom :focus-visible styling to any interactive elements (button, links, input, textarea). This will make the users can navigate this website using keyboard (by using Tab key) easily.

  I hope this helps! Keep up the good work! 👍

  13

  Marked as helpful
• 

  Deepak Parmar•490

  @deepak-parmar

  Posted over 3 years ago

  What is the best way to deal with Page should contain a level-one heading warning? I kept my <h1> heading hidden, but it didn't work.

  13
• 

  Andres Gonzalez•130

  @AndyGlz01

  Posted over 3 years ago

  Is perfect!!! Just add some margin-top, height of the cards!

  00