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

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

@nvaccessAuto
Copy link

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.

@nvaccessAuto
Copy link
Author

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.

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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

@nvaccessAuto
Copy link
Author

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).

@nvaccessAuto
Copy link
Author

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.

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

1 participant