Order Summary Component w/ CSS Grid and custom CSS properties

Community feedback

• 

  Fer•3,970

  @fernandolapazPosted over 1 year ago

  Hi 👋, just a couple of things in case you want to take a look:

  HTML / ACCESSIBILITY:

  • The icon is a decorative image and therefore needs an empty alt attribute to be ignored by a screen reader.

  CSS:

  • You might consider using relative units like rem or em since they are better for scalable layouts. Something simple to start with would be to convert to rem (1 rem equals the font size of the root element, 16px by default). Consider this suggestion especially for the font-size.

  I hope it’s useful : )

  Nice solution btw.

  Regards,

  Marked as helpful

  0

  ---

  

  newtothis90•240

  @newtothis90Posted over 1 year ago

  @fernandolapaz

  That was very helpful! Especially the icon being a decorative image and having an empty alt attribute.

  Thank you very much for taking the time to give me your advice! It's very appreciated.

  1