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

nvda does not support dropdown menus and lists in excel 2010 #3382

Closed
nvaccessAuto opened this Issue Jul 24, 2013 · 19 comments

Comments

Projects
None yet
1 participant
@nvaccessAuto

nvaccessAuto commented Jul 24, 2013

Reported by gregjozk_ on 2013-07-24 13:19
When working in excel documents (example is attach here) nvda does not announce dropdown menus, lists and content of that elements. see column c. this file is using on system windows 7 32-bit nvda next branch and office 2010.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Jul 24, 2013

Attachment nvdalogandexamplefile.zip added by gregjozk_ on 2013-07-24 13:25
Description:
In the zip file is nvda log and xlsm file where is dropdown.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 19, 2013

Attachment dropdown.xls added by jteh on 2013-08-19 05:13
Description:
Simpler test case. Choices are in a1:a4, drop-down is in b1.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 21, 2013

Comment 1 by Michael Curran <mick@... on 2013-08-21 10:28
In [f303b93]:

Initial support for dropdown lists in MS Excel. Should support 2003 through 2010, relying on displayModel. Re #3382

Currently looks for white text for selected item. should of course change this to use proper system selection color.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 22, 2013

Comment 2 by Michael Curran <mick@... on 2013-08-22 01:38
In [a5b5175]:

Excel dropdowns: now uses the proper system highlight colors to detect selection. Also include some code that should have been committed before. Re #3382.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 23, 2013

Comment 3 by Michael Curran <mick@... on 2013-08-23 12:44
In [951c417]:

Excel dropdowns: now uses the proper system highlight colors to detect selection. Also include some code that should have been committed before. Re #3382.

Note this support covers Excel 2003 2007 and 2010, but not 2013 sadly.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 23, 2013

Comment 4 by Michael Curran <mick@... on 2013-08-23 12:44
In [02d8431]:

Merge branch 't3382' (Support Excel 2003-2010 dropdown lists) into next. Incubates #3382

Changes:
Added labels: incubating

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 23, 2013

Comment 5 by Michael Curran <mick@... on 2013-08-23 12:47
In [5e51f25]:

Merge branch 't3382' into next. Incubates #3382

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Aug 28, 2013

Comment 6 by jteh on 2013-08-28 03:23
Changes:
Milestone changed from None to next

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Sep 12, 2013

Comment 7 by Michael Curran <mick@... on 2013-09-12 01:45
In [e6409e6]:

Merge branch 't3382'. Fixes #3382

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Sep 12, 2013

Comment 8 by mdcurran on 2013-09-12 01:46
Changes:
Milestone changed from next to 2013.3

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 16, 2013

Comment 9 by gregjozk_ on 2013-10-16 06:10
I see, that in the fresh edition of next branch (see attached zip file), dropdown does not work again.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 16, 2013

Comment 10 by gregjozk_ on 2013-10-16 06:13
Changes:
State: reopened

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 16, 2013

Attachment 1310 JG - Časovnica.zip added by gregjozk_ on 2013-10-16 06:16
Description:
log + file when I observed a problem.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 17, 2013

Comment 11 by mdcurran (in reply to comment 9) on 2013-10-17 05:24
Replying to gregjozk_:
I am able to open and navigate the dropdown on c19 in your example. Did this ever work for you? What version of MS Excel are you running?
This feature is also now in master. Do you have any success trying this with master?

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 17, 2013

Comment 12 by gregjozk_ on 2013-10-17 05:57
I'm using excel version 2010 (32-bit) on os windows 7 sp1. Problem is, because sometimes dropdown works, but often NVDA does not recognise, that in c (or whatever column there is dropdown and then it does not open dropdown options.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 18, 2013

Comment 13 by mdcurran on 2013-10-18 01:31
For anyone who can't seem to get dropdowns to work in Excel when pressing alt+downArrow, please test this try build:
http://community.nvda-project.org/try/excelDropdownTimeout/nvda_snapshot_try-excelDropdownTimeout-9641,ef5ec7c.exe

Let me know if this works better.
If it still does not work for you, please provide a log set to input/output. A part from some improvements to detection of the Dropdown window, there is also extra logging to show where it fails.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 18, 2013

Comment 14 by gregjozk_ on 2013-10-18 06:48
it seems, that this solution works fine. thanks. the excel dropdown (yours yesterday submition) works well and main next branch makes trouble.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 18, 2013

Comment 15 by Michael Curran <mick@... on 2013-10-18 06:55
In [f3e21f1]:

Merge branch 'excelDropdownTimeout' into next. Incubates #3382

Changes:
Added labels: incubating

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Oct 31, 2013

Comment 16 by Michael Curran <mick@... on 2013-10-31 23:38
In [967990b]:

Merge branch 'excelDropdownTimeout'. Fixes #3382.

Changes:
Removed labels: incubating
State: closed

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