Three Card Component using CSS

Community feedback

• 

  P

  Dave•5,295

  @dwhensonPosted over 3 years ago

  Hi @hariramjp777 very nice work on this one! 🙌

  Not much that I can suggest for improving things, some very small suggestions are:

  1. Perhaps try using text-transform: uppercase for the h2 elements rather than typing the actual letters in uppercase - a very small deal!! - but a good habit to get into in case you need to change the design later.

  2. Another tiny point, you probably don't need alt text for the icons as they aren't really descriptive content. I would probably just leave the "" empty, which means that screen readers will just skip the element and go to the headings.

  These are really small points though!! Keep up the great work and happy coding! 👍 Cheers 👋

  1

  ---

  

  Hari Ram•370

  @hariramjp777Posted over 3 years ago

  Hi, @dwhenson Thanks for your feedback. Cheers

  0