1186876 - Context menu doesn't hide when I click twice outside of it (if panel was opened)

Status: NEW

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[arni2033]

Attachment: Context menus should close after clicking on devTools panel - testpage.htm

STR:
[video] https://dl.dropboxusercontent.com/s/7w8fulil3caxsz9/Context%20menus%20should%20close%20after%20clicking%20on%20devTools%20panel%20-%20screencast.webm?dl=0
1.   Open attachment page
2.   Inspect <div> element
3.   Open any context menu
4.A) Doubleclick "asdf" in ruleview pane
4.B) Doubleclick src of <img>

RESULT: Context menu hasn't closed

Expectations: context menu should've closed

Comment 1

[arni2033]

I should note that it's only reproducible if you wait until preview popup will show up, and only then actually doubleclick.

Comment 2

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Hm, I can't reproduce this on Windows or Mac with the current Nightly.
Could you try this with a clean profile to make sure that it's not the fault of an add-on?

Comment 3

[arni2033]

(In reply to Philipp Sackl [:phlsa] (Firefox UX) please use needinfo from comment #2)
> Hm, I can't reproduce this on Windows or Mac with the current Nightly.
> Could you try this with a clean profile to make sure that it's not the fault of an add-on?
Just reproduced on latest Nightly 42.0a1 (2015-07-26) with new profile. It's presented for long time actually. Have you followed the note in Comment 1? (popup with image preview or "could not find image")

Comment 4

[[:Towkir] Ahmed]

Strange, it opened a new tab (after clicking on "asdf") but the context menu still did not close.
Reproducible each time.

Anyway, is this happening because of devetool or contex menu ? I am talking about the component

Comment 5

[arni2033]

(In reply to Towkir Ahmed [:Towkir] from comment #4)
> Anyway, is this happening because of devetool or contex menu ? I am talking
> about the component
This is happening because popups in devtools are placed on top of everything. So your click event just gets cancelled (I guess). This is reproducible not only with context menus, but also with opened select drop-down lists. I'll attach another test page if I have time.

Comment 6

[[:Towkir] Ahmed]

Yeah, I have seen the case of drop down lists, arni, you can see bug 1180641 about that

Comment 7

[arni2033]

I meant another thing: When you click, then the eventListener in devtools (I assume) hides that popup and does something like event.stopImmediatePropagation(); and therefore the eventListener for hiding context menus on click - just isn't called.
The real implementation may be different but it behaves this way.

Comment 8

[arni2033]

D.E.T.A.I.L.E.D. STR:

1.   Open attachment 8637867 [details] (attached to this bug)
2.   Inspect <div> element on that page
3.   Right-click on the empty place on the page (white area) to open content area context menu
4.A) Hover mouse over "asdf" in ruleview pane
4.B) Hover mouse over the value of <img>'s src (in markup)
 [do NOT move mouse after Step 4]
5.   DoubleClick left mouse button

AR:
  A) Page "https://bug1186876.bmoattachments.org/asdf" loads in a new foreground tab.
!    Context menu opened in Step 3 is still visible.
  B) Textarea appears in markup-view to allow editing text
    «src="chrome://browser/skin/customizableui/[e-mail]"».
     Text "chrome://browser/skin/customizableui/[e-mail]" is selected in that textarea.
!    Context menu opened in Step 3 is still visible.

ER:
 Context menu should close after first click in Step 5. Or at least after the second click in Step 5

Comment 9

[Patrick Brosset <:pbro>]

Inspector bug triage (filter on CLIMBING SHOES).

Comment 10

[arni2033]

>>>   My Info:   Win7_64, Nightly 49, 32bit, ID 20160526082509
STR_2:
1. Open attached "testcase 1" in non-e10s window
2. Right-click in input field on the page
3. Hover mouse over the button on the page
4. Doubleclick the button on the page

AR:  Context menu stays on page
ER:  Context menu should hide after 2nd click for sure

Comment 11

[arni2033]

Attachment: testcase 1 (bug 1186876) - doubleclick doesn't hide context menu.xul

Comment 12

[Neil Deakin]

I can see this only on Windows and only if you double-click rather than just click twice. You can easily just click again to close it.

This is an obscure edgecase scenario though.

Comment 13

[arni2033]

(In reply to Neil Deakin from comment #12)
> I can see this only on Windows and only if you double-click rather than just
> click twice. You can easily just click again to close it.
> 
> This is an obscure edgecase scenario though.

Please confirm that you want to keep context menu appearing from irrelevant tab as long as possible
I'm sick of "edge cases", they happen too often. (I also added NI request because nobody else including me seem to figured the right component)

[OOT]
There's also not so "edge case" in bug 1171394 that you (team) are equally ignoring. For 17+ month.
Is "edge case" property of a bug a real reason or you just pull it out when it applies?
It's just interesting for my what is the reason for you (team) to ignore all those breakages in UI in favor of "improvements" of searchbar, locationbar, new bookmark panel etc. that only break UX.

Comment 14

[Vlad Bacia-Mociran [:VladB]]

Attachment: issue video

User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0

I have verified this issue using the scenario from comment 0 on latest Nightly (Build ID: 20170103030204) and it is reproducible with just clicking on background URL after hovering it for 1-2 seconds.