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(ui5-card): correctly set aria-labelledby #3692

Merged
merged 1 commit into from Aug 19, 2021
Merged

Conversation

dimovpetar
Copy link
Contributor

Now card doesn't reference element ID in another shadow DOM with aria-labelledby. The header is labelled by its own title instead.

Fixes #3643

@dimovpetar dimovpetar requested a review from a team August 19, 2021 08:16
Copy link
Contributor

@fifoosid fifoosid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, someone from @SAP/ui5-webcomponents-topic-rd should approve and we can merge it

@ilhan007 ilhan007 merged commit 1e57b00 into master Aug 19, 2021
@ilhan007 ilhan007 deleted the fix_card_labelling branch August 19, 2021 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ui5-card][A11y]: Title refrence is not valid
4 participants