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

Override SelectInternal to add UIA_AutomationFocusChangedEventId in PropertyGridView.GridViewTextBox #12479

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LeafShi1
Copy link
Member

@LeafShi1 LeafShi1 commented Nov 14, 2024

Fixes #12421

Proposed changes

  • Override SelectInternal in PropertyGridView.GridViewTextBox to add RaiseAutomationEvent(UIA_EVENT_ID.UIA_AutomationFocusChangedEventId)

Customer Impact

  • When switching the items by using up/down keyboard arrow that without expand the dropdown list panel, Narrator announced the property item entirety.

Regression?

  • Yes

Risk

  • Minimal

Screenshots

Before

When switching the items by using up/down keyboard arrow that without expand the dropdown list panel, Narrator can not announced the property item entirety.
image

After

When switching the items by using up/down keyboard arrow that without expand the dropdown list panel, Narrator announced the property item entirety.
image

Test methodology

  • Manually

Test environment(s)

  • .net 10.0.0-alpha.1.24562.13
Microsoft Reviewers: Open in CodeFlow

@LeafShi1
Copy link
Member Author

There have extra "item selected" is announced for BackColor/ForeColor/Cursor property with Narrator and the issue cannot reproduce with NVDA tools

image

Copy link

codecov bot commented Nov 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.74259%. Comparing base (af49f16) to head (76e5ac3).

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #12479         +/-   ##
===================================================
+ Coverage   75.73242%   75.74259%   +0.01017%     
===================================================
  Files           3153        3153                 
  Lines         635807      635814          +7     
  Branches       46975       46976          +1     
===================================================
+ Hits          481512      481582         +70     
+ Misses        150870      150802         -68     
- Partials        3425        3430          +5     
Flag Coverage Δ
Debug 75.74259% <100.00000%> (+0.01017%) ⬆️
integration 18.28657% <100.00000%> (+0.03204%) ⬆️
production 49.32756% <100.00000%> (+0.02203%) ⬆️
test 97.05107% <ø> (+0.00113%) ⬆️
unit 46.26686% <42.85714%> (+0.00379%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant