Skip to content

Excel - Listing the comments in the current sheet fails #11366

@CyrilleB79

Description

@CyrilleB79

Steps to reproduce:

  • Open a new Excel document
  • In cell A1, press shift+F2, enter a dummy comment and validate
  • In cell A3, press shift+F2, enter a dummy comment and validate
  • Press NVDA+F7 to open the element list dialog.
  • shift+tab to turn back to radio buttons indicating the element type
  • Press right or left arrow to land on the comment radio button
  • press tab to go to the comments list

Actual behavior:

The error beep is heard and nothing is found in the comment's list. The following error is found in the log:

Input: kb(desktop):rightArrow
ERROR - RPC process 6636 (EXCEL.EXE) (11:21:26.916) - Dummy-7635 (13692):
Thread 356, build\x86\remote\excel.cpp, nvdaInProcUtils_excel_getCellInfos::<lambda_252607b36749a934dcf7252ca0219d42>::operator (), 478:
application.range failed. Code -2146827284, rangeAddress [Classeur1]Feuil1!$A$2,$A$4

IO - speech.speak (11:21:26.936) - MainThread (11308):
Speaking ['Commentaires', 'bouton radio', 'coché', 'Alt+o']

Note: this error does not occur if the comments are on contiguous cells.

Expected behavior:

No error and the both comments are listed.

System configuration

NVDA installed/portable/running from source:

Installed and portable

NVDA version:

NVDA 2020.2beta2

Windows version:

Windows 10 1809

Name and version of other software in use when reproducing the issue:

Excel 2016

Other information about your system:

Other questions

Does the issue still occur after restarting your computer?

Yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

Yes:
2019.1.1: the issue is still present.
2019.1: All is working well.

If addons are disabled, is your problem still occuring?

Yes

Did you try to run the COM registry fixing tool in NVDA menu / tools?

Yes

Additional notes:

I have not not been able to find which commit has caused this regression since my environment does not seem to allow me to runn NVDA from source from this time.
Anyway, it semms that the following commit is a good candidate for this regression:
commit e194b89
Author: Michael Curran michaelDCurran@users.noreply.github.com
Address crash in Excel 2007 (#9456)

If this first commit is not responsible for the regression, here is another commit that may interfer:
commit 0df2a49
Author: Takuya Nishimoto nishimotz@gmail.com
Excel cell range text should be translated. #9470 (#9471)

@michaelDCurran may you have a look since you are the author of e194b89?
Thanks.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions