Initial support for Windows 10 anniversary versions of both Microsoft Edge and Windows 10 Mail.#6271
Merged
Conversation
…icknav to work correctly in Edge browse mode.
* GetChildren / GetEnclosingElement zoom to the deepest control at the start. Therefore we must take this into acocunt.
…r the root element.
…e end that are incorrectly placed there by Edge (E.g. expanding to line before a list, table etc includes the list or table).
…thus we no longer need to blindly allow all UIA focus events on the root container. This also allows remembering of the caret position when alt tabbing away and back again.
…here appropriate.
… linked, bold, italic, underline, strike through, super/subScript).
…ue per object by adding UIA's runtime ID to the controlField. E.g. Arrowing down from one list to another at the same level use to class the controlField for each list as being the same list.
…t is not needed and stops us from correctly chopping the extra elements accideintally included by Edge at the end of lines.
…on't need to collect information about their root element at all. In deed, _getTextWithFields_unbalanced shuold *not* recurse after its root element. Stops an infinit loop when dealing with aria-hidden text.
…d switch to focus mode.
…ng with quicknav. UIATextInfo._getControlFieldForObject: Restore old behaviour for name.
…operty if available.
… is not equal to name.
…fetching landmarks.
…phics, if they can be invoked.
…n embedded child (checkbox etc) at the beginning. This stops skipping out of the document when a control is inside a label, and standardizes this check with similar checks in other parts of the code.
…_getTextWithFields_unbalanced which allows for a substantial removal of duplicate code.
…tributeValue rather than findAttribute. Using findAttribute was never totally accurate in all situations, and its speed was based on the size of the document rather than how far the next heading was. In most cases moving by heading is now faster. In a few cases it may have become slower, though it is now more in line with Narrator.
…s, and the unit is either line or paragraph or the reason is focus, always announce headings in intial formatting even if the heading level has not changed. Edge exposes its headings via formatting and many users are confused as to why headings are not announced again when repeting the same line like in other browsers.
…nt in certain situations.
…ntrol such as as a checkbox does not include the control in its children or enclosing element. Also ensure these controls are focused with using quicknav / arrowing.
… to help braille presentation.
jcsteh
approved these changes
Sep 23, 2016
…he next line when expanding to line/paragraph. Other code already handled 90% of usecases. This code fixed one and introduced one. For now minimal is better.
Contributor
|
These last commits look fine to me. |
Contributor
|
hi all. when reading text in edge with arrow keys, small chunks are displayed on each line compared to ff. is it something that should be improved? |
…TextRange::getEnclosingElement is not implemented (File Explorer search, Win 7 start menu search) Fixes #6411
michaelDCurran
added a commit
that referenced
this pull request
Sep 29, 2016
Contributor
|
Hi, Perhaps a consistency problem: In Edge, you can use browse mode to move between lists but not list items. STR:
Expected: You can navigate between list items. Setup:
Thanks. |
* UIATextInfo: When fetching formatted text, firstly try fetching formatting for the entire chunk, and only falling back to splitting by unit if a mixed attribute is found. Also, allow subclasses to override the UIA format units to be tried when splitting. * Catch COMError when trying to fetch the selection with IUIAutomationTextRange::getSelection and raise RuntimeError as expected by the rest of NVDA. Fixes #6454
Member
Author
jcsteh
approved these changes
Oct 18, 2016
| @ type formatConfig: a dictionary of NVDA document formatting configuration keys with values set to true for those types that should be fetched. | ||
| @param UIAFormatUnits: the UI Automation text units (in order of resolution) that should be used to split the text such that formatting won't have mixed values. | ||
| @type UIAFormatUnits: List of UI Automation Text Units | ||
| @param UIAFormatUnits: the UI Automation text units (in order of resolution) that should be used to split the text such that formatting won't have mixed values. If None, then self.UIAFormatUnits will be used. |
Contributor
There was a problem hiding this comment.
This needs to be clarified to specify the behaviour for the three cases: a list of units, None or an empty list of units.
| unit=UIAFormatUnits[0] | ||
| furtherUIAFormatUnits=UIAFormatUnits[1:] | ||
| else: | ||
| unit=None |
Contributor
There was a problem hiding this comment.
Can we have a comment here saying that this will try the entire range first before any units?
michaelDCurran
added a commit
that referenced
this pull request
Oct 19, 2016
michaelDCurran
added a commit
that referenced
this pull request
Nov 2, 2016
michaelDCurran
added a commit
that referenced
this pull request
Nov 2, 2016
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #6292.
Fixes #5987.
Fixes #6168.