Amon• 2,560
@A-amon
Posted
Hello! It looks great. So is the responsiveness~ 😀
Here are few suggestions:
- Wrap your site's main content into a
<main>
tag. 🏠 - Have a
<h1>
heading. Probably something like this:
<h1 class="sr-only">Order summary component</h1>
Note: The .sr-only
class is to hide it from view but still available for screen reader. Here is one way to do it. 😉
- Use
<button>
or<a>
for your 'Proceed to payment" and "Cancel" elements. - After fixing the
<h1>
, you can read up on heading levels if you getHeading levels should only increase by one
accessibility issue 👌
2