Ahmed Bayoumi• 6,800
@Bayoumi-dev
Posted
Hey!
Congratulations on completing this challenge... You have accessibility issues
that need to fix.
Buttons must have the discernible text
, So don't leave thealt
attribute empty when using the image<img>
inside the button<button>
without text.
<button class="absolute left-2/4 bg-neongreen p-4 rounded-[50%] -translate-x-1/2 hover:shadow-neongreen" id="generate-advice">
<img src="./images/icon-dice.svg" alt="Advice Generator">
</button>
Or set the attribute aria-label
to describe the button.
Page should contain a level-one heading
, Change<div class="text-xs text-center uppercase text-neongreen text-[10px] tracking-[.25em]">
to<h1 class="text-xs text-center uppercase text-neongreen text-[10px] tracking-[.25em]">
You should always have oneh1
per page of the document... in this challenge, you will useh1
just to avoid theaccessibility issue
that appears in the challenge report... but don't useh1
on small components<h1>
should represent the main heading for the whole page, and for the best practice use only one<h1>
per page.- An
img
element must have analt
attribute, to provide alternative information for an image if a user for some reason cannot view it, but in this case, add an emptyalt
attribute to avoid the accessibility issue because it is an unnecessary icon for the content.
<img src="./images/pattern-divider-desktop.svg" alt="" class="hidden md:block">
I hope this is helpful to you... Keep coding👍
1
harshj11• 50
@harshj11
Posted
@Bayoumi-dev thank you so much for the suggestions I would surely work on them.
0