@mattstuddert
Posted
Nice work Muhammad! I've just got a couple of small recommendations, so nothing major:
- I would lowercase the "Ping" logo in your HTML so that screen reader software doesn't read it out letter-by-letter. You can then use
text-transform: uppercase;
in your HTML to visually uppercase it. - You've gone straight in with a
h3
heading, but you've not got anyh1
orh2
elements. I would always recommend starting with ah1
as your main page title.
I hope this helps. Keep up the great work! 👍
Marked as helpful
@mattstuddert very helpful thanks for the feedback. one question how to solve the problem in my report? I have been looking for it but have not found the right solution
@mattstuddert
Posted
@fadzrinmadu you need to either add some text inside the anchor tag, which you can then visually hide so that it's only accessible to screen reader users. Or you can add an aria-label
attribute to the anchor tags to provide the link text.
@mattstuddert thank you for your answer