Ensure outlook AutoCompleteListItems are highlighted when selected#18484
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an issue where the NVDA Highlighter was not highlighting Outlook recipient auto-complete entries when selected. The solution adds vision handler integration and navigator object updates to ensure proper highlighting behavior when users navigate through auto-complete suggestions.
- Adds vision handler integration to handle focus events for auto-complete list items
- Updates navigator object to maintain proper focus tracking for highlighting
This sounds like a potentially blocking issue. Why isn't this the case? |
The auto-complete list and its items are not focusable and never receive system focus. The only events the By setting the NVDA-internal focus and nagivator object to these list items, the Highlighter highlights them correctly. That is what I am doing so far. The edit field themselves receive If you can think of a way to direct focus back to the edit field when the list is dismissed, I would be more than happy to try to implement it. |
How are you validating what events are fired? With MSAA/Events debugging categories? |
Currently I am overriding all I know that the winEvent |
|
|
It would make sense to do something similar to what happens in other auto-complete lists. Looking at the start menu, when I type in the search field and then press arrows, the focus and nav object highlighters move between suggestions. Pressing On the opposite, the experience with Firefoxaddress bar seems similar to what is described in this PR. In any case, the fact that the highlighter does not turn back to the edit field when the suggestions are dismissed is problematic IMO. |
|
I have spent quite some time investigating the events fired on the auto-complete list and its items, as well as on the parent elements of the list. Unfortunately, I could not find any event, like The only solution that I could think of, that works, is to detect the I have also looked at the general Highlighter behavior on the Windows start menu. There it seems that only the navigator object is set to the selected item while the focus object remains on the edit field. If this is the desired behavior, I can adjust my code to also only set the navigator object. |
At first glance, seems nice to me. Are there drawbacks to this solution? Or uncovered use cases / workflows?
Here it should be discussed and agreed to know which is the desired behaviour? Start menu behaviour or Firefox address bar? Then you can change your code according to what is chosen. |
None that I can currently think of. |
@Nerlant a few that spring to mind:
|
When the user clicks on another element that can have focus, the highlight rectangle moves to the element with focus.
As soon as the value of the contact edit field changes, the focus is set back to it. This applies both to entering text in the edit field and to accepting suggestions. |
Even if this focusable element is the field where they have typed the first characters? In any case, even if there are such corner cases, in my opinion the benefit is higher than the little bugs introduced by the corner cases (if any). |
When clicking into the edit field for which the auto-complete list is currently opened, the list closes for a short moment before re-opening. The highlighter rectangle then resets to the first entry of the auto-complete list, which is automatically selected by Outlook. |
|
Please add a changelog entry under bug fixes |
Co-authored-by: Sean Budd <seanbudd123@gmail.com>
|
@Nerlant - can you please resolve merge conflicts? or grant me permission to push to your branch |
I will resolve them and request a review from you again |
Link to issue number:
Closes #18483
Summary of the issue:
The NVDA Highlighter does not highlight Outlook recipient auto-complete entries
Description of user facing changes:
The NVDA Highlighter highlights Outlook recipient auto-complete entries correctly
Description of developer facing changes:
Description of development approach:
Testing strategy:
Known issues with pull request:
Code Review Checklist:
@coderabbitai summary