-
-
Notifications
You must be signed in to change notification settings - Fork 784
Edit controls in explorer fail when they have a native UIA implementation #15375
Copy link
Copy link
Closed
Labels
bug/regressionfeature/windows-explorerWindows start menu, explorer, and other in-box functionality.Windows start menu, explorer, and other in-box functionality.p2https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityhttps://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#prioritytriagedHas been triaged, issue is waiting for implementation.Has been triaged, issue is waiting for implementation.
Milestone
Metadata
Metadata
Assignees
Labels
bug/regressionfeature/windows-explorerWindows start menu, explorer, and other in-box functionality.Windows start menu, explorer, and other in-box functionality.p2https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityhttps://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#prioritytriagedHas been triaged, issue is waiting for implementation.Has been triaged, issue is waiting for implementation.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Regression from #15314
Steps to reproduce:
Actual behavior:
Read only edit boxes don't read. This is because they have an overlay in the explorer appModule that is bound to IAccessible rather than Edit.
Expected behavior:
Edit boxes read correctly.
System configuration
NVDA version:
2023.3 Beta branch
Windows version:
Windows 11 22H2
Additional details
The code conflicting with UIA was introduced in 32a0f22, pr #8643. Cc @lukaszgo1
Personally I'm tempted to remove this RTL and LTR workaround from the explorer appModule. While this will reintroduce these marks in the said edit boxes, removing the marks from the windowText is no viable solution to me as it will cause of by one errors when reporting the symbol or character under the caret.