504163 - In download progress dialog, indicate availability of context menus through WAI-ARIA attribute.

Status: RESOLVED FIXED

(SeaMonkey :: Download & File Handling, defect)

Description

[Marco Zehe (:MarcoZ)]

In the download progress dialog, indicate that the file name and server name labels have context menus that can be invoked. Use the aria-haspopup attribute with a value of "true" on these elements, as specified here:
http://www.w3.org/TR/wai-aria/#aria-haspopup

Comment 1

[Marco Zehe (:MarcoZ)]

Attachment: Patch

Comment 2

[[e-mail]]

So accessibility ignores the context and popup attributes?

Comment 3

[alexander :surkov (:asurkov)]

(In reply to comment #2)
> So accessibility ignores the context and popup attributes?

Good point. Afaik we don't expose any relations between context menu and element to AT and AT users invokes keyboard shortcuts like sighted users do. But iirc we have a bug to expose context menu as well. Concerning popup menus, yes, I think we should take into account @popup attribute and expose STATE_HAS_POPUP on related element.

Comment 4

[Marco Zehe (:MarcoZ)]

(In reply to comment #3)
> yes, I think we should take into account @popup attribute and expose
> STATE_HAS_POPUP on related element.

I filed bug 504252 for this to deal with it in trunk. However since we don't know whether this will make its way into 1.9.1 eventually, I suggest taking this as a workaround, and reverting it as soon as SeaMonkey regularly builds against trunk and we have bug 504252 fixed.

Comment 5

[alexander :surkov (:asurkov)]

(In reply to comment #4)

>I suggest taking
> this as a workaround, and reverting it as soon as SeaMonkey regularly builds
> against trunk and we have bug 504252 fixed.

I don't mind though it's worth to add XXX comment to refer new bug.

Comment 6

[[e-mail]]

Comment on attachment 388527 [details] [diff] [review]
Patch

I don't need an XXX comment here if you note the link between the bugs here so that you remember to undo this at some point after fixing the other bug ;-)

Comment 7

[[e-mail]]

That is, if you note (presumably in bug 504252) to revert this change.

Comment 8

[Marco Zehe (:MarcoZ)]

Comment on attachment 388527 [details] [diff] [review]
Patch

Cancelling approval since it's past code freeze for B1. Waiting for tree to reopen.

Comment 9

[Marco Zehe (:MarcoZ)]

Pushed: http://hg.mozilla.org/comm-central/rev/e800384a1890