Skip to content

refactor: replaces aria-hidden with alt=""#25911

Merged
crisbeto merged 1 commit intoangular:mainfrom
dgp1130:autocomplete-alt
Nov 3, 2022
Merged

refactor: replaces aria-hidden with alt=""#25911
crisbeto merged 1 commit intoangular:mainfrom
dgp1130:autocomplete-alt

Conversation

@dgp1130
Copy link
Contributor

@dgp1130 dgp1130 commented Nov 2, 2022

An alt attribute is required for an upcoming extended diagnostic, and from my understanding is desirable over aria-hidden since it maintains role="img" while also indicating that the image is decorative.

See CI failures from: https://app.circleci.com/pipelines/github/angular/angular/52667/workflows/3d58122f-0257-4b79-9ab9-90475590a3f2/jobs/1249004

@dgp1130 dgp1130 requested a review from andrewseguin as a code owner November 2, 2022 17:38
@dgp1130 dgp1130 added the target: patch This PR is targeted for the next patch release label Nov 2, 2022
An `alt` attribute is required for an upcoming extended diagnostic, and from my understanding is desirable over `aria-hidden` since it maintains `role="img"` while also indicating that the image is decorative.
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@crisbeto crisbeto added merge safe action: merge The PR is ready for merge by the caretaker labels Nov 2, 2022
@crisbeto crisbeto merged commit b260cb1 into angular:main Nov 3, 2022
crisbeto pushed a commit that referenced this pull request Nov 3, 2022
An `alt` attribute is required for an upcoming extended diagnostic, and from my understanding is desirable over `aria-hidden` since it maintains `role="img"` while also indicating that the image is decorative.

(cherry picked from commit b260cb1)
crisbeto pushed a commit that referenced this pull request Nov 3, 2022
An `alt` attribute is required for an upcoming extended diagnostic, and from my understanding is desirable over `aria-hidden` since it maintains `role="img"` while also indicating that the image is decorative.

(cherry picked from commit b260cb1)
@dgp1130 dgp1130 deleted the autocomplete-alt branch November 3, 2022 16:55
dgp1130 added a commit to dgp1130/angular that referenced this pull request Nov 3, 2022
This updates the components CI test to use the latest commit on their `main` branch (angular/components@a999a52). This is needed to test with angular/components#25911, which fixes an issue introduced by the new extended diagnostic.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants