Skip to content

Announce live region changes with high priority speech#10622

Closed
LeonarddeR wants to merge 1 commit into
nvaccess:masterfrom
LeonarddeR:liveRegionHighPrio
Closed

Announce live region changes with high priority speech#10622
LeonarddeR wants to merge 1 commit into
nvaccess:masterfrom
LeonarddeR:liveRegionHighPrio

Conversation

@LeonarddeR

Copy link
Copy Markdown
Collaborator

Link to issue number:

None

Summary of the issue:

Live region changes are currently reported at normal speech priority.

Description of how this pull request fixes the issue:

Change the priority to high to ensure that live region messages are queued before any other speech.

Testing performed:

T.b.d.

Known issues with pull request:

None

Change log entry:

  • Changes
    • Live region changes, mainly fired by UIA, are now announced with higher speech priority.

@feerrenrut

Copy link
Copy Markdown
Contributor

This should probably take into account and reflect the POLITENESS_SETTING See docs

@LeonarddeR

Copy link
Copy Markdown
Collaborator Author

You are correct. However, for that to be supported there's a lot to be done.

  1. We need to fetch the UIA LiveSetting property and map that to a property that can be used to decide what speech prio to use.
  2. The IA2 live handler needs to be changed to take priority into account like this. For this, we also need to expose choosing speech priority in NVDAHelper.

I think it makes sense to close this pull request for now, and open a new one with a complete implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants