Update UIA alert event in role mapping table to match UIA changes #229
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Actually, we will change this behavior in Chrome. I discussed your idea of not firing alerts when the content of the alert is empty with James Craig and Aaron this morning and they both agreed it would be a welcomed improvement to the spec. I'll implement this change in Chromium. I filed https://issues.chromium.org/issues/337315124 and https://issues.chromium.org/issues/337315123 to change the Chromium behavior. Thanks again! |
|
[heart] Doug Geoffray reacted to your message:
…________________________________
From: Benjamin Beaudry ***@***.***>
Sent: Friday, April 26, 2024 11:44:55 PM
To: w3c/core-aam ***@***.***>
Cc: Doug Geoffray ***@***.***>; Review requested ***@***.***>
Subject: Re: [w3c/core-aam] Update UIA alert event in role mapping table to match UIA changes (PR #229)
Actually, we will change this behavior in Chrome. I discussed your idea of not firing alerts when the content of the alert is empty with James Craig and Aaron this morning and they both agreed it would be a welcomed improvement to the spec. I'll implement this change in Chromium.
I filed https://issues.chromium.org/issues/337315124 and https://issues.chromium.org/issues/337315123 to change the Chromium behavior. Thanks again!
—
Reply to this email directly, view it on GitHub<#229 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AM4H76NJZJIZ42RHXZPMMXTY7LRHPAVCNFSM6AAAAABGZTRODOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGIZTSMZUGU>.
You are receiving this because your review was requested.Message ID: ***@***.***>
|
|
Discussed this internally, and instead of going this route Edge is going to revert to alert event mappings but not fire events for empty alerts. |
|
[heart] Doug Geoffray reacted to your message:
…________________________________
From: Sarah Higley ***@***.***>
Sent: Tuesday, May 7, 2024 5:58:17 PM
To: w3c/core-aam ***@***.***>
Cc: Doug Geoffray ***@***.***>; Review requested ***@***.***>
Subject: Re: [w3c/core-aam] Update UIA alert event in role mapping table to match UIA changes (PR #229)
Discussed this internally, and instead of going this route Edge is going to revert to alert event mappings but not fire events for empty alerts.
—
Reply to this email directly, view it on GitHub<#229 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AM4H76MLT4OHMOEOBJ3FQUTZBEI3TAVCNFSM6AAAAABGZTRODOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJZGAYDENRVGA>.
You are receiving this because your review was requested.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on screen reader feedback, the
alertrole no longer fires a system alert event, but a live region changed event (still on insertion).Related Chromium PR with the change: https://chromium-review.googlesource.com/c/chromium/src/+/3715513
Preview | Diff