- Issue created by @malcomio
- π¬π§United Kingdom aaron.ferris
aaron.ferris β made their first commit to this issueβs fork.
- Merge request !10Issue #3466209: only add role img if the svg has a title β (Merged) created by aaron.ferris
- Status changed to Needs review
5 months ago 9:42am 10 August 2024 - π¬π§United Kingdom aaron.ferris
Added a condition to use the role and aria label should a title be defined. If not, no role, no title and no aria label.
- π¬π§United Kingdom the_g_bomb
I am not sure that removing the `role="img"` from svgs that do not also contain a title, necessarily make that module more accessible.
I suspect we may need to also remove the item from he screen readers output to avoid confusion.
Without a role img it may pass check for missing alt text, but it seems to me to be a workaround of the testing tool.
Perhaps a adding `aria-hidden="true"` would help. ref: https://www.unimelb.edu.au/accessibility/techniques/accessible-svgs
- π¬π§United Kingdom aaron.ferris
Yes thats a good idea, pushed a change.
-
aaron.ferris β
committed 96c9d5b1 on 8.x-1.x
Issue #3466209: only add role img if the svg has a title
-
aaron.ferris β
committed 96c9d5b1 on 8.x-1.x
- Status changed to Fixed
5 months ago 3:56pm 16 August 2024 - Status changed to Fixed
5 months ago 3:58pm 16 August 2024