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 cannot support extendable button in this page. #4667

Closed
nvaccessAuto opened this Issue Dec 4, 2014 · 4 comments

Comments

Projects
None yet
2 participants
@nvaccessAuto

nvaccessAuto commented Dec 4, 2014

Reported by vgjh2005 on 2014-12-04 11:22
Hi:
The browser is internet explorer 11:
Step to reproduce:
1.Open the webpage: http://detail.aliqin.tmall.com/item.htm?spm=a1z2p.7404678.1998151914.4.WeZW4R&id=39990583842#mod=buy
2.Press b to switch to the first extendable button;
3.Press spacebar to activate the button;
4.Press down arror to select the menu items then press enter to select current one
prospective:Read each selected item and make a selection.
actually:Button cannot be extended correctly.
Thanks!

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Dec 4, 2014

Comment 1 by jteh on 2014-12-04 23:07
This works as expected in Firefox. It doesn't work in IE 11 because of a bug in IE 11's accessibility implementation. I'm not sure whether we can work around this yet.

Technical: IE 11 doesn't expose the focused state when aria-activedescendant is used, though it does fire a focus event. I don't think we want to just stop checking the focused state altogether for IE. I guess we could walk up the parents looking for aria-activedescendant, but that's pretty slow.

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Feb 5, 2015

Comment 3 by Michael Curran <mick@... on 2015-02-05 01:04
In [dd80fe5]:

Merge branch 't4667' into next. Incubates #4667

Changes:
Added labels: incubating

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Feb 9, 2015

Comment 4 by Michael Curran <mick@... on 2015-02-09 06:42
In [5932460]:

Merge branch 't4667'. Fixes #4667

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto

This comment has been minimized.

nvaccessAuto commented Feb 9, 2015

Comment 5 by mdcurran on 2015-02-09 06:43
Changes:
Milestone changed from None to 2015.1

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