Open
Bug 726453
Opened 13 years ago
Updated 2 years ago
[meta] Improve accessibility and keyboard interaction
Categories
(Firefox :: Downloads Panel, defect)
Firefox
Downloads Panel
Tracking
()
NEW
People
(Reporter: Paolo, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug, )
Details
(Keywords: access, meta)
This bug tracks work to be done on top of the Downloads Panel in bug 726444.
* Allow cycling through the "show download history" entry using the arrows.
* Focus usually returns to where one was before download manager when pressing
Escape. However, if nothing was focused when DM was invoked, focus should be
set explicitly to the document, so that it doesn't get into a limbo state.
* The option to open the file in focus, available using the ENTER key, is not
announced. Understand how to address this when using screen readers.
One thing I've noticed on the keyboard interaction side, if I open the download panel (from clicking the download button or using Ctrl-J), the "Show all downloads" part of the panel has the "S" underlined for a keyboard shortcut. But it won't do anything until you click somewhere inside the panel to give the panel content focus. (To do this without accidentally activating the link in the panel, click-and-hold over the "Show all downloads" part and drag up past the upper edge of the panel, then release the mouse button. This works for me on Windows, not sure about other platforms.)
Reporter | ||
Updated•13 years ago
|
Blocks: DownloadsPanel, ReleaseDownloadsPane
Shouldn't this have been a "tracked" bug way back in February? It's rather unfortunate to ship with an accessibility regression like this.
tracking-firefox17:
--- → ?
Reporter | ||
Comment 3•12 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #2)
> Shouldn't this have been a "tracked" bug way back in February? It's rather
> unfortunate to ship with an accessibility regression like this.
The feature is disabled by default in all channels except Nightly, thus for now
we're only using track flags to ensure the feature is actually disabled and we
have no regressions while disabled.
To track things we need to fix before shipping, we're using bug 747422, and not
the tracking flags, because we don't know in which release the panel will be
ready to be enabled for everyone.
tracking-firefox17:
? → ---
Comment 4•12 years ago
|
||
Should the issues in comment 0 be broken down into individual bugs blocking bug 747422?
Flags: needinfo?(mak77)
Comment 5•12 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #4)
> Should the issues in comment 0 be broken down into individual bugs blocking
> bug 747422?
yes that'd be good, likely once those are fixed we also need sort of a review from accessibility on the interaction.
Flags: needinfo?(mak77)
Comment 6•12 years ago
|
||
Actually, considered we changed Downloads to open the DM and not the panel, exactly to help blind users accessibility, could make the latter issue not even a blocker. The panel is mostly considered a mouse target at this point, even if common keyboard access would still be welcome as it is for any other widget in the toolbar.
Comment 7•12 years ago
|
||
* Focus usually returns to where one was before download manager when pressing
Escape. However, if nothing was focused when DM was invoked, focus should be
set explicitly to the document, so that it doesn't get into a limbo state.
What does limbo state mean? which part of the code this affects? If nothing was focused what's wrong to go back to the same state? How do we open the panel without focusing?
I basically filed all the other bugs reported here but this one, Paolo could you please do that?
No longer blocks: ReleaseDownloadsPane
Flags: needinfo?(paolo.mozmail)
Reporter | ||
Comment 8•12 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #7)
> * Focus usually returns to where one was before download manager when
> pressing Escape. However, if nothing was focused when DM was invoked, focus
> should be set explicitly to the document, so that it doesn't get into a
> limbo state.
This was in turn copied from bug 564934 comment 366, I'm redirecting the
information request to Marco Zehe.
Marco, do you know if the "limbo state" you mention still affects the version of
the panel that is currently in Nightly? And I quote Marco's comments:
What does limbo state mean? which part of the code this affects? If nothing was focused what's wrong to go back to the same state? How do we open the panel without focusing?
Flags: needinfo?(paolo.mozmail) → needinfo?(marco.zehe)
Comment 9•12 years ago
|
||
I believe the eefault is to focus the document now if there's no previously focused item. The mechanism should be that, if there's a previously focused item, after the panel closes, focus should return to it. The user should not be put into a state where the item after leaving the panel is different from the one she was on before opening it. I believe that's what I meant with my original quoted comment.
Flags: needinfo?(marco.zehe)
Comment 10•12 years ago
|
||
Re my last comment: Restoring focus to the previously focused item is already working fine in nightly, both if the focus is on a page element or on the document itself.
Comment 11•12 years ago
|
||
Thanks for the clarification Marco.
So our task is basically to verify we don't steal focus when closing the panel.
Updated•12 years ago
|
Component: General → Downloads Panel
Updated•12 years ago
|
No longer blocks: DownloadsPanel
Updated•12 years ago
|
Updated•12 years ago
|
Keywords: meta
Summary: [Downloads Panel] Improve accessibility and keyboard interaction → Improve accessibility and keyboard interaction
Updated•2 years ago
|
Severity: normal → S3
Updated•2 years ago
|
Summary: Improve accessibility and keyboard interaction → [meta] Improve accessibility and keyboard interaction
You need to log in
before you can comment on or make changes to this bug.
Description
•