Stats preview card component

Community feedback

• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted almost 3 years ago

  Hello PRANAV , I have some suggestions:

  • Numbers don't make sense as h1s.

  • The number and word have to be read together to make sense so need to be in the same meaningful element (list item) . so only a <span> or maybe <strong> tag needs to wrap the numbers. .And the words like companies should not be in paragraph tags. They don't need to be wrapped in anything as they are already inside a meaningful element (list item)

  • Don't capitalise in html, let css text transform take care of that. Remember screen readers won't be able to Read capitalised text as they will often read them letter by letter thinking they are acronyms.

  • You can apply border radiusand overflow hidden to the component(.Content).

  • Your solution isn't responsive.

  Hopefully this feedback helps.

  Marked as helpful

  0