Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

single-price-grid-component

Achilike Justice• 250

@ArchieK9

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

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 for width and height. Also, you don't need to set height 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 GitHub
Discord logo

Join 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