3 Column Preview Card Component

Community feedback

• 

  Ken•4,915

  @kens-visuals

  Posted almost 4 years ago

  Hey @IshitaBisaria 👋🏻

  I have a quick tip for the icons. I suggest adding aria-hidden="true", because they're for decoration only. For example, <img src="images/icon-luxury.svg" alt="" aria-hidden="true" class="logo">. You can read more about aria-hidden here.

  I hope this was helpful 👨🏻‍💻 overall, you did a great job, just tweak media-query sizes a bit. That's it from me, cheers 👾

  02

  Marked as helpful
• 

  Fidel Lim•2,775

  @fidellim

  Posted almost 4 years ago

  Hi Ishita,

  Great work finishing the project! Just some suggestions you can apply:

  • you can add cursor:pointer to the buttons so that the user knows that it can be clicked.
  • for your media query, it would be better to increase is as some of the component is hidden. you can set the media query to @media (min-width: 900px) instead.

  I hope it helps. :)

  03

  Marked as helpful
• 

  Muhammad Fadll•320

  @fadelun

  Posted almost 4 years ago

  great, no issues detected

  03

  Marked as helpful