added __ne__ to textInfos.TextInfo class#5991
added __ne__ to textInfos.TextInfo class#5991mohammad-suliman wants to merge 2 commits intonvaccess:masterfrom
Conversation
|
also, the new line script bounded to enter doesn't work after this change. |
|
Just to clarify, I assume you mean this only breaks for UIA? Or were you
seeing it break elsewhere as well?
|
|
Yes, with the text edit fields you recommended to test withVS text editorSearch field of start menu Get Outlook for iOS On Tue, May 24, 2016 at 3:43 PM -0700, "James Teh" [email protected] wrote: Just to clarify, I assume you mean this only breaks for UIA? Or were you seeing it break elsewhere as well? — |
|
Regarding backspace, we think expanding to character before creating the bookmark in EditableText._backspaceScriptHelper might fix this. This needs testing, though. I'm not sure why enter isn't working. Does it just say nothing (instead of the start of the line)? That issue seems strange to me because the bookmarks before and after enter is sent should definitely be different because the cursor has moved, and we're adding to the text, not deleting, so there's no ambiguity. |
|
if I remember correctly, I concluded it doesn't work depending on prints On 5/31/2016 9:01 AM, James Teh wrote:
|
|
unfortunately, it didn't work, I did the following:
On 5/31/2016 9:01 AM, James Teh wrote:
|
Ug. It fixes the issue for the search box in Explorer, but not the search box in the Win 10 Start Menu. |
…ether the caret has moved
|
Jamie, why we can't use TextSelectionChanged event to determine whether the caret has moved as mentioned in this link under "Interoperability with the System Caret" heading: |
|
Yeah, Mick and I discussed this too. We previously listened for that event. Unfortunately, some controls (I believe WPF 2.5 text controls, but I'd need to hunt down specifics) flood those events, causing major performance problems. There's no way for us to block events for specific controls. Welcome to the agony of UI Automation and it's myriad of stupidly broken implementations, one of the great banes of my existence. |
|
Fixing this seems to be a not easy challenge! |
|
Could you please update the initial description of this pull request according to the pull request template? |
|
Adding blocked label since there are open issues for which we don't yet have solutions. See comments above, especially #5991 (comment) and #5991 (comment). |
|
I'm going to close this PR, there has not been any meaningful activity in a long time, and it's not clear what the initial intention was. |
I tested NVDA's behavior after the change: unfortunately, backspace script no longer works. no errors in the log, the char before the caret is deleted, but NVDA doesn't make any announcement