Single price grid component

Community feedback

• 

  Vanza Setia•27,715

  @vanzasetia

  Posted over 2 years ago

  Hi, Seranela! 👋

  Nice improvements! It's great that you take another step to fix the color-contrast issues. Good job! 👍

  Here are some suggestions for improvements.

  • I recommend replacing the second and the third h1 with h2. It is not a best practice to have many h1 on a page. It can confuse the screenreader users because there will be many titles for a single page.
  • I recommend reading the "How-to: Accessible heading structure - The A11Y Project" article.
  • Never limit the height of the body element. It will not allow the users to scroll the page if the page content needs more height. You can see the issue by looking at the site on a mobile landscape view. So, my recommendation is to use min-height instead.

  Overall, it looks great. It's amazing that you make the site more accessible. 👏

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

  02

  Marked as helpful