Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When NVDA is set to read columns in Excel columns aren't read if there is separation between column headers and data. #5396

Closed
nvaccessAuto opened this issue Oct 1, 2015 · 7 comments
Assignees
Milestone

Comments

@nvaccessAuto
Copy link

Reported by csm120 on 2015-10-01 23:32
I've seen this behavior in Excel 2010, 2013, and 2016.
Steps:

  1. In Excel make 3 columns from a1 to c3.
  2. Type data starting in a3.
  3. Use NVDA+Shift+c on a1.
  4. Reading of columns is spotty or not happening unless you delete the blank row at a2.
    I have tried this with NVDA 2015.3, and with latest snaps.
    I need the columns to be read even if there is separation between the column headers and the data. I'm attaching a file.
@nvaccessAuto
Copy link
Author

Attachment NVDA Columns.xlsx added by csm120 on 2015-10-01 23:33
Description:
Excel spreadsheet with columns set to read in NVDA

@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2015-10-02 09:16
This is because Excel defines a blank row or column to indicate a new region. In order to facilitate headers for different regions, NVDA honours this and only reports headers within the region.

That said, this seems to be causing a lot of confusion and I wonder whether we need to drop the region support. The reality is that regardless of whether Excel defines these as new regions, users (rightly or wrongly) don't seem to care.

@nvaccessAuto
Copy link
Author

Comment 2 by bramd on 2015-10-26 22:34
The region support is useful in some situations. We might "extend" the search for headers to regions left/above of the current region if the current region doesn't define headers? This might be what users expect and still allows for per region headers. That being said, this is just from a functional perspective and I don't know if there are any technical issues preventing this.

@jcsteh
Copy link
Contributor

jcsteh commented Jan 27, 2016

This was incubated months ago in b283199, but the automated script didn't catch it for some reason.

There are a couple of additional things that are needed for this:

  • If one defines two sets of headers, the first set will be read as the headers when moving over the second set of headers. For example, if a1 and c1 are set as column headers, a1 will be read as a header when moving from c2 to c1. We shouldn't search for a header in the same axis for a cell which is a header in that axis itself.
  • NVDA 2014.3 RC1 not reading a specific Excel spreadsheet #4409 fixed an issue which caused exceptions in a protected sheet if reporting of headers was enabled. I think (but am not sure) that this just disables headers, rather than just disabling region checks. That case needs to be checked.

michaelDCurran added a commit that referenced this issue Mar 8, 2016
…ger limited to an Excel region. First try of #5396.

This reverts commit dbfe229.
@nvaccessAuto
Copy link
Author

Incubated in c7ec64c.

@nvaccessAuto
Copy link
Author

Incubated in 6637f7f.

@jcsteh
Copy link
Contributor

jcsteh commented May 6, 2016

Note: c14f16e in next on 12 April reverted 35f0f0d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants