Design comparison
SolutionDesign
Community feedback
- @vanzasetiaPosted almost 2 years ago
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
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord