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

Already on GitHub? Sign in to your account

No indication about the selection with CTRL+A is given, on selecting a set of data in a excel sheet that is different and is separated from the data present in the same sheet. #3043

Closed
nvaccessAuto opened this Issue Mar 4, 2013 · 9 comments

Comments

Projects
None yet
1 participant

Reported by sumandogra on 2013-03-04 11:23
Excel 2010
If there are groups of data in a sheet separated by blank rows and columns, we can select a set of data that can be defined under a name range with CTRL+A. This selection is not announced by NVDA.
Steps to reproduce:
Open a workbook that has groups of data arranged and separated by blank rows and columns.
Go to a set of data that is separate from any set of data on the sheet and press CTRL+A.
No indication is given by NVDA that the set of data has been selected.

for an example pls see the attached sheet.

Attachment Example sheet for different sets of data.xlsx added by sumandogra on 2013-03-04 11:28
Description:
just an example sheet to show the presence of different sets of data present in an excel sheet.

Comment 2 by vrdhn on 2013-09-18 11:41
Hi,
The Fix of this is to add 'Control+A' to the list of selection change gestures in excel.py.
The changes can be picked from:
Link to DIFF: https://bitbucket.org/manish_agrawal/nvda/commits/413c63a1bda00107c1c2522bf2728afce49a3476#chg-source/NVDAObjects/window/excel.py

Repository: https://vrdhn@bitbucket.org/manish_agrawal/nvda.git
BRanch: t3043

Thanks
Vardhan

Comment 4 by Michael Curran <mick@... on 2013-10-01 06:48
In [7295458]:

Excel worksheet NVDAObject: control+a should be treeted as a selection change. Re #3043

Comment 5 by Michael Curran <mick@... on 2013-10-01 06:48
In [3926b55]:

Merge branch 't3043' into next. Incubates #3043

Changes:
Added labels: incubating

Comment 6 by mdcurran on 2013-10-01 06:49
Changes:
Milestone changed from None to next

Comment 7 by Michael Curran <mick@... on 2013-10-15 09:43
In [2d22641]:

Merge branch 't3043'. Fixes #3043

Changes:
Removed labels: incubating
State: closed

Comment 8 by mdcurran on 2013-10-15 09:43
Changes:
Milestone changed from next to 2013.3

Comment 9 by ragb on 2013-11-24 12:25
Just noticed a small problem with the approach taken on this ticket. When office is in portuguese (probably other languages too) the key bindings are changed: control+a stands for open and control+t for select-all. We may solve this using a locale gestures file, however, the control+a should be unbound somehow.

Shall I open another ticket? (don't think this blocks the 2013.3 release).

Comment 10 by jteh on 2013-11-25 02:16
Damn. I was always worried about something like this happening. Binding scripts to keys like this is ugly, but there really isn't a reliable event we can use here. Using locale bindings will work for most, but it breaks if the user has different NVDA and Ms Office languages.

Btw, you can unbind a key using a locale gesture map (just specify the script as None), so that isn't a problem.

Please do open another ticket for this and mention that it was spun off this one. Thanks.

@nvaccessAuto nvaccessAuto added this to the 2013.3 milestone Nov 10, 2015

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