Skip to content

Fixing the incorrect ScreenReader blue rectangle for a new character when typing (Text pattern area)#3969

Merged
RussKie merged 1 commit intodotnet:masterfrom
vladimir-krestov:Issue_3897_FixingNarratorRectange_for_TypedCharacters
Oct 12, 2020
Merged

Fixing the incorrect ScreenReader blue rectangle for a new character when typing (Text pattern area)#3969
RussKie merged 1 commit intodotnet:masterfrom
vladimir-krestov:Issue_3897_FixingNarratorRectange_for_TypedCharacters

Conversation

@vladimir-krestov
Copy link
Copy Markdown
Contributor

@vladimir-krestov vladimir-krestov commented Sep 21, 2020

Fixes #3897

Proposed changes

  • Raise Text_TextChangedEvent to Narrator shows the correct blue rectangle when typing new characters in Edit controls (should work as in Win32 RichTextBox)
  • Do not raise the event for RichTextBox because Win32 provides unmanaged Text pattern for it

Customer Impact

  • A user orientates on the blue rectangle when entering text, and the current behavior can be confusing. This fix implements expected behavior for all text controls as in RichTextBox

Regression?

  • No

Risk

  • Minimal

Screenshots

Before

  • Narrator doesn't show the blue rectangle correctly
    TT bug

After

  • The correct blue rectangle
    T

Test methodology

  • CTI
  • Manual testing

Accessibility testing

  • Using Narrator

Test environment(s)

  • .Net 5.0 Version: 5.0.0-rc.1.20431.5
  • Microsoft Windows [Version 10.0.19041.450]
Microsoft Reviewers: Open in CodeFlow

@vladimir-krestov vladimir-krestov requested a review from a team as a code owner September 21, 2020 21:50
@vladimir-krestov vladimir-krestov added this to the 5.0 RC2 milestone Sep 21, 2020
@vladimir-krestov vladimir-krestov added waiting-for-testing The PR is awaiting manual testing by the primary team; no action is yet required from the author(s) waiting-review This item is waiting on review by one or more members of team labels Sep 21, 2020
@RussKie RussKie modified the milestones: 5.0 RC2, 6.0, 6.0 Preview1 Sep 21, 2020
@RussKie
Copy link
Copy Markdown
Contributor

RussKie commented Sep 21, 2020

We had a discussion and decided this doesn't meet the Bar.
Please rebase on top of the master.

@RussKie RussKie added waiting-author-feedback The team requires more information from the author and removed waiting-review This item is waiting on review by one or more members of team labels Sep 21, 2020
@vladimir-krestov vladimir-krestov changed the base branch from release/5.0-rc2 to master September 22, 2020 08:28
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Sep 22, 2020
@vladimir-krestov vladimir-krestov force-pushed the Issue_3897_FixingNarratorRectange_for_TypedCharacters branch from fa1d076 to 7a2f25c Compare September 22, 2020 08:33
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 22, 2020

Codecov Report

Merging #3969 into master will decrease coverage by 31.16633%.
The diff coverage is 100.00000%.

@@                 Coverage Diff                  @@
##              master       #3969          +/-   ##
====================================================
- Coverage   67.65118%   36.48485%   -31.16633%     
====================================================
  Files           1418         934         -484     
  Lines         505901      250123      -255778     
  Branches       41060       36764        -4296     
====================================================
- Hits          342248       91257      -250991     
+ Misses        157693      153633        -4060     
+ Partials        5960        5233         -727     
Flag Coverage Δ
#Debug 36.48485% <100.00000%> (-31.16633%) ⬇️
#production 36.48485% <100.00000%> (+0.01060%) ⬆️
#test ?

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

RussKie
RussKie previously approved these changes Sep 23, 2020
@vladimir-krestov vladimir-krestov removed the waiting-for-testing The PR is awaiting manual testing by the primary team; no action is yet required from the author(s) label Sep 29, 2020
@vladimir-krestov vladimir-krestov marked this pull request as draft September 29, 2020 14:26
@vladimir-krestov vladimir-krestov added the ⛔ blocked Blocked by external dependencies label Sep 29, 2020
@vladimir-krestov
Copy link
Copy Markdown
Contributor Author

Text pattern is not implemented for master branch so this fix is blocked because it is based on Text patter implementation.

@vladimir-krestov vladimir-krestov removed the ⛔ blocked Blocked by external dependencies label Oct 2, 2020
@vladimir-krestov vladimir-krestov marked this pull request as ready for review October 5, 2020 10:16
@vladimir-krestov vladimir-krestov added the waiting-review This item is waiting on review by one or more members of team label Oct 5, 2020
@vladimir-krestov
Copy link
Copy Markdown
Contributor Author

Testers approved these changes ✔️
There are 2 non-regression ComboBox issues. They will be placed in a separate Issue.

@vladimir-krestov vladimir-krestov force-pushed the Issue_3897_FixingNarratorRectange_for_TypedCharacters branch from 7a2f25c to 3fe64d8 Compare October 5, 2020 10:21
@vladimir-krestov vladimir-krestov force-pushed the Issue_3897_FixingNarratorRectange_for_TypedCharacters branch from 3fe64d8 to 2be0418 Compare October 5, 2020 10:38
@RussKie RussKie merged commit 7fab08b into dotnet:master Oct 12, 2020
@RussKie RussKie removed the waiting-review This item is waiting on review by one or more members of team label Oct 12, 2020
@vladimir-krestov vladimir-krestov deleted the Issue_3897_FixingNarratorRectange_for_TypedCharacters branch November 4, 2020 07:35
@ghost ghost locked as resolved and limited conversation to collaborators Jan 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Accessibility] Incorrect ScreenReader blue rectangle for a new character in TextCompoments

2 participants