P
Jairo Valderrama• 420
@jairovg
Posted
Hi @Saguneo, congrats on your solution; here are some comments that might help you to improve it:
- Even if the challenge is a single component and you probably won't have a page in a real project with just one of it, think about how to improve the component's semantics. May it have a main title? Probably a subtitle in any particular section? Just to give you an example: You have some recurrent categories for the summary section, is the
<div>
element the best one to use for each category? MDN docs have a guide named Content Categories that may help you with this. - You have multiple
a11y
issues; you can use tools like axe dev tools if you're not using them yet. - If you run an
a11y
audit you will notice 9 serious issues, 7 of then related to colour contrast that are easily fixed on this challenge, but on a daily basis may be handled with the UX designer, and another one that is"Ensures interactive controls are not nested as they are not always announced by screen readers or can cause focus problems for assistive technologies"
, related to theanchor
you have within thebutton
you can read more about this issue on the Deque University Axe rules page.
I hope you find it useful. I'm happy to take another look at your solution if you make some other changes.
0