Add missing formatting information in MS Excel cells (strikethrough, superscript/subscript)#12264
Merged
Merged
Conversation
…superscript/subscript)
See test results for failed build of commit 3ea39bcef6 |
Contributor
Author
|
The failing test is Robot.chromeTests.ARIA treegrid |
See test results for failed build of commit ac7c06ef12 |
Contributor
Author
|
Oh, I have closed this PR by mistake and then re-opened it. This seems to trigger a new AppVeyor build. Good to know... Now, another system test is failing: Robot.chromeTests.i7562. |
See test results for failed build of commit ccae80cdb0 |
See test results for failed build of commit 23be5e7cbe |
bcd5f7b to
b1a5026
Compare
seanbudd
approved these changes
Apr 20, 2021
Member
There was a problem hiding this comment.
Thanks @CyrilleB79 for this, and for your patience dealing with our randomly failing system tests.
Here's the excel file I used to test with
test-12264.xlsx
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:
None
Summary of the issue:
Some formatting information is not reported when navigating from one cell to another in Excel, although the corresponding category is checked in Document formatting settings.
Description of how this pull request fixes the issue:
_getFormatFieldAndOffsetsreportFocus, speak the format field sequence not only if the current cell has format field info to speak but also if the previous cell has it. This is done to report 'baseline' when transitioning from a cell with superscript to a cell without formatting since 'baseline', being the default, is not stored in the cell's format fields.Note
This PR allows to have strikethrough and superscript/subscript reported if the whole cell is formatted with these attributes.
If having a whole cell formatted as superscript or subscript is probably not a real-life use case, strikethrough may be quite common.
Testing strategy:
Manual test:
No specific unit or system test for this PR.
Known issues with pull request:
This PR does not address the following points that are still issues with NVDA:
Change log entry:
Section: Bug fixes
Strikethrough, superscript and subscript are now reported when navigating through Excel cells if the corresponding option is enabled (#12264)Code Review Checklist:
This checklist is a reminder of things commonly forgotten in a new PR.
Authors, please do a self-review and confirm you have considered the following items.
Mark items you have considered by checking them.
You can do this when editing the Pull request description with an x:
[ ]becomes[x].You can also check the checkboxes after the PR is created.