Closed Bug 1115964 Opened 10 years ago Closed 7 years ago

Context menu items like "Remove from history" and "Open containing folder" fail to work, and the download loses its highlight


(Firefox :: Downloads Panel, defect)

22 Branch
Not set



Tracking Status
firefox34 --- wontfix
firefox35 --- wontfix
firefox36 --- wontfix
firefox37 --- wontfix
firefox-esr31 --- wontfix


(Reporter: nicolas.barbulesco, Unassigned)



(Keywords: regression)


(2 files)

Firefox 34.0.5 Mac, on Mac OS X 10.6.8. 
I have a download that is finished. 
I click on the blue arrow. 
The Downloads bubble appears. It has one download. 
I place my mouse over my download. It becomes blue. 
I right-click my download. 
I choose "Remove from history". 

* Expected result : * 
The download was blue before the right click. It remains blue during the whole right click, wherever the mouse goes. When I release the right click, the download goes away.

* Actual result : * 
During the right click, when the mouse goes away from the download, the download becomes back white. When I release the right click, the download remains. I try again… Finally, at the 5th try, the removal works. It seems that, for the removal to work, the mouse must be over the download when I release the right click — which is difficult because of the height of the contextual menu.

Thank you for correcting this bug.

Nicolas B.
I can reproduce on Windows7
Ever confirmed: true
OS: Mac OS X → All
About #1 command "Remove from history" fails to work:

Regressed by:
63c003375416	Marco Bonardo — Bug 851519 - Keyboard commands may executed on the wrong download in the panel r=Mano
Blocks: 851519
Flags: needinfo?(mak77)
Keywords: regression
Version: unspecified → 22 Branch
So the gesture you are trying is right-button down on a download and release on the Remove From History contextual menuitem?
Does it work if you just right click on the download and then left click on Remove From History?
Points: --- → 3
Flags: qe-verify+
Flags: needinfo?(mak77)
Flags: firefox-backlog+
1. Download something
2. Left mouse click download toolbar button so that download panel pops up
3. Right mouse click a entry of download (you should click the right-bottom part of the highlighted area so that context menu does not overlap with highlighted area as much as possible) 
4. Left click "Remove From History" by mouse (mouse pointer should be positioned outside of the arrow panel)

Actual Results:
"Remove From History" does nothing.
"Open containing folder", "Go To Download Page" and "Copt Download Link" also fail.
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150419030206

Mozilla/5.0 (X11; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150419030206

On Windows 7, the downloaded item remained highlighted. It only lost its highlight in Linux Mint KDE 17.1.
Summary: The contextual menu item "Remove from history" fails to work, and the download loses its blue colour → Context menu items like "Remove from history" and "Open containing folder" fail to work, and the download loses its highlight
Comment on attachment 8821934 [details]
Bug 1115964 - Inhibit download item selection if a context menu is open

::: browser/components/downloads/content/downloads.js:1032
(Diff revision 1)
>    /**
>     * Mouse listeners to handle selection on hover.
>     */
>    onDownloadMouseOver(aEvent) {
> -    if ( == this.richListBox) {
> +    if (!this.contextMenuOpen &&
> + == this.richListBox) {

I might have slightly messed up the submission here. Lines 1032 and 1033 here are relative to, rather than the original repo.

If you were to apply this changeset, you'd have to apply that one first.
This is related to bug 991965.
Attachment #8821934 - Flags: review?(paolo.mozmail)
Paolo asked in bug 991965 (comment 17 there) to better explain what problem my patch for this bug solves:

In the Downloads Panel, if you open a context menu for an item (by right-clicking it), you can still hover over and select other items. In this case, when you click 'Remove From History', you may (unintuitively) remove an item different than the one you opened the context menu on.

The attached screencast shows selection behaviour with an open context menu before and after the patch.
Comment on attachment 8821934 [details]
Bug 1115964 - Inhibit download item selection if a context menu is open

Reviewed this in bug 991965 comment 19.

::: browser/components/downloads/content/downloads.js:1030
(Diff revision 1)
>    onDownloadMouseOver(aEvent) {
> -    if ( == this.richListBox) {
> +    if (!this.contextMenuOpen &&
> + == this.richListBox) {

As noted in the other bug, we should fix the incorrect use of originalTarget in this function too.

::: browser/components/downloads/content/downloadsOverlay.xul:63
(Diff revision 1)
> +                  onpopupshown="DownloadsView.onContextPopupShown(event);"
> +                  onpopuphidden="DownloadsView.onContextPopupHidden(event);"

nit: there's an extra space, ensure these are aligned correctly.
Attachment #8821934 - Flags: review?(paolo.mozmail)
Over in bug 991965, we noted that when the context menu is open, clicking anywhere else just closes the menu. Thus, we might suppress the normal mouse hover highlight in the Downloads Panel while the context menu is open. Instead, we could replace it with an item-wide highlight, which will be "locked" to the item that was right-clicked originally.

Carol, any preference for how this highlight could look like, or any recommendation against implementing it? I'm thinking of using the current dimmed style, but across the entire download item.

This is conceptually identical to what the system file managers do, although on Windows 7 the highlight for the clicked item looks more like a mouse hover highlight, while on Mac OS X 10.9 it looks more like a focus ring.

Let me know if anything is unclear!
Flags: needinfo?(chuang)
Hi Paolo,
I checked other items on the menu panel, once the context menu is open, the hover highlight will only on the mouse-over item. Therefore, in my opinion, to be consistent, we could keep the current behavior. We don't need item-wide highlight or other highlight for the item was right-clicked originally.
I hope it makes sense.
Let me know if you have any question or something unclear! thank you :)
Flags: needinfo?(chuang) → needinfo?(paolo.mozmail)
I see what you mean, for example the tab strip has the same issue. If I invoke the context menu on any tab that is not selected, then there is no persistent indication of which tab I invoked the menu on.

This is possibly something to improve in general in the user interface of the browser, but I'll close this bug that is specific to the Downloads Panel for now. If we fix the issue with the more important user interface elements like the tab strip, then we can revisit this for the Downloads Panel as well.
Closed: 7 years ago
Flags: needinfo?(paolo.mozmail)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.