Skip to content

Excel cell border announcement #3044#6492

Closed
nishimotz wants to merge 6 commits into
nvaccess:masterfrom
nishimotz:i3044-excel-cellborder
Closed

Excel cell border announcement #3044#6492
nishimotz wants to merge 6 commits into
nvaccess:masterfrom
nishimotz:i3044-excel-cellborder

Conversation

@nishimotz

Copy link
Copy Markdown
Contributor

Information about the cell border is given in an Excel sheet, if the cell border is different from the adjacent cells. #3044

@michaelDCurran michaelDCurran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get the following error when manually reporting formatting with NVDA+f when on an cell in Excel:

ERROR - scriptHandler.executeScript (10:49:44):
error executing script: <bound method GlobalCommands.script_reportFormatting of <globalCommands.GlobalCommands object at 0x05DA40F0>> with gesture u'NVDA+f'
Traceback (most recent call last):
  File "scriptHandler.py", line 187, in executeScript
    script(gesture)
  File "globalCommands.py", line 1287, in script_reportFormatting
    for field in info.getTextWithFields(formatConfig):
  File "textInfos\offsets.py", line 392, in getTextWithFields
    field,(boundStart,boundEnd)=self._getFormatFieldAndOffsets(self._startOffset,formatConfig,calculateOffsets=False)
  File "NVDAObjects\window\excel.py", line 963, in _getFormatFieldAndOffsets
    if formatConfig["reportBorderStyle"]:
KeyError: 'reportBorderStyle'

Other than that this looks good.

…cell border is different from the adjacent cells. NVDA+F should work on Excel as well. nvaccess#3044 PR nvaccess#6492
…cell border is different from the adjacent cells. NVDA+F should work on Excel as well. Now merged master and resolved confict regarding configSpec. nvaccess#3044 PR nvaccess#6492
@feerrenrut

Copy link
Copy Markdown
Contributor

This PR is superseded by PR #6895, which will be re-incubated to fix regression #6889. Closing this PR now.

@feerrenrut feerrenrut closed this Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants