Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(VImg): pass alt prop to img element #16841

Merged
merged 2 commits into from
Mar 13, 2023

Conversation

ffxsam
Copy link
Contributor

@ffxsam ffxsam commented Mar 5, 2023

Description

<v-img> alt prop was not being passed down to the underlying <img> tag. It was only being set on a higher level as an aria-label, which is not sufficient for a11y requirements. Fixes #16203.

@johnleider johnleider requested a review from KaelWD March 6, 2023 19:42
@KaelWD KaelWD added T: enhancement Functionality that enhances existing features C: VImg VImg labels Mar 8, 2023
@KaelWD KaelWD added this to the v3.1.x milestone Mar 8, 2023
@KaelWD KaelWD changed the title Pass alt prop to <img> fix(VImg): pass alt prop to <img> element Mar 8, 2023
@KaelWD
Copy link
Member

KaelWD commented Mar 13, 2023

Both NVDA and TalkBack read this the same as before (just fine) but ok.

@KaelWD KaelWD changed the title fix(VImg): pass alt prop to <img> element fix(VImg): pass alt prop to img element Mar 13, 2023
@KaelWD KaelWD merged commit d09d70b into vuetifyjs:master Mar 13, 2023
@ffxsam ffxsam deleted the fix/img-alt-attr branch March 13, 2023 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VImg VImg T: enhancement Functionality that enhances existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug Report][3.0.0] v-img alt text prop support is missing
2 participants