Vanza Setia• 27,795
@vanzasetia
Posted
Hi, Achilike Justice! 👋
I have a few suggestions for improving this solution.
- Headings must be in a logical order. Users of assistive technology can use heading tags to navigate the website. If headings are not in a logical order, they can get confused.
- There should not be more than one
h1
on a page. Many<h1>
elements mean many titles which can confuse the users, especially the screen reader users. - Dive deeper — How-to: Accessible heading structure - The A11Y Project
- The content below the "Why Us" text should a list of bullet points.
- Use one media query to switch from a one-column layout into a grid layout. The card only needs a
max-width
—no need forwidth
andheight
. Also, you don't need to setheight
on the other elements. Let the content controls the height of it.
I hope my feedback helps. 🙂
0