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

Consider mapping accessible name change events #291

Closed
mfairchild365 opened this issue May 28, 2020 · 2 comments · Fixed by #507
Closed

Consider mapping accessible name change events #291

mfairchild365 opened this issue May 28, 2020 · 2 comments · Fixed by #507
Labels
accName & Desc Related to accessible name or description

Comments

@mfairchild365
Copy link

Core-AAM provides mappings for name change events that occur via aria-label and aria-labelledby.

It might be worthwhile to map HTML based name changes to appropriate events in HTML-aam. For example, the <label> of a form control, or the innerText of a <button> or <a href>.

@scottaohara
Copy link
Member

so this is mapped in core aam

MSAA + IAccessible2 event
EVENT_OBJECT_NAMECHANGE

UIA event
PropertyChangedEvent
Property for aria-label: AriaProperties
Property for aria-labelledby: LabeledBy

ATK/AT-SPI event
object:property-change:accessible-name

I think what likely needs to be done here is this needs to be specifically called out at the start or end of of a general naming algorithm (there is a lot of redundancy that could be cut down here) to indicate that if the name changes, to ensure that changed name is exposed via the platform's event for this.

scottaohara added a commit that referenced this issue Oct 6, 2023
closes #291 

adding note to reference the already specified name/desc change events in core aam.
@cookiecrook
Copy link
Collaborator

cookiecrook commented Oct 13, 2023

FYI noticed this while reviewing the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accName & Desc Related to accessible name or description
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants