Fix revert in 5b821c60a - #12965
Merged
Merged
Conversation
A rename of field to textWithEmbeddedObjectsItem was not observed when reverting. Re-introduce renames.
Contributor
Author
|
@michaelDCurran the case from #12766 will need to be retested. |
michaelDCurran
approved these changes
Oct 21, 2021
Member
|
I have confirmed that the Google Docs case in #12766 is still fixed. The Thunderbird case is very hard to reproduce, but I'll have much more of a chance to see it once running this code on beta or master for an extended period of time. Though I am quite confident the Thunderbird and Google Docs cases were the same bug. |
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.
Link to issue number:
Fixes #12919
Summary of the issue:
When using NVDA with Libre / Open office Writer, text from the document can not be read.
In #12500 the variable used to iterate over
_iterTextWithEmbeddedObjectswas changed fromfieldtotextWithEmbeddedObjectsItem, additionallychildObjectwas used instead of field forti.obj.getChild(embedIndex)In #12766 some of these changes were reverted, however
textWithEmbeddedObjectsItemwas no longer appended tofieldsinsteadfieldwas.Description of how this pull request fixes the issue:
Re-introduce renames.
Testing strategy:
Manually tested with OpenOffice:
The content should be read, no error sound should be heard.
Known issues with pull request:
None
Change log entries:
None
Code Review Checklist: