Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(tooltip): use different attribute to track setting of aria-label #10600

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

kara
Copy link
Contributor

@kara kara commented Apr 19, 2017

We recently started setting aria-labelledby on parent elements to track if the aria-label had already been set by the tooltip. It seems we should use a different attribute as a marker because aria-labelledby will take precedence over aria-label when set on the same element. In this case, it will point to an element that only exists on the DOM when the tooltip is showing, which hurts accessibility. This is currently blocking 1.1.4 release due to Google breakages.

cc: @ThomasBurleson

@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label Apr 19, 2017
@kara kara requested a review from jelbourn April 19, 2017 21:33
@jelbourn
Copy link
Member

LGTM

@jelbourn jelbourn added the pr: merge ready This PR is ready for a caretaker to review label Apr 19, 2017
@kara kara merged commit 471c225 into angular:master Apr 19, 2017
@ernsheong
Copy link
Contributor

@kara can you help with the PR here #10537 as well? thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants