"Disable Popup Auto-Hide" not working when trying to inspect overflowing "All Tabs" list panel popup
Categories
(DevTools :: Inspector, defect)
Tracking
(firefox-esr91 unaffected, firefox100 unaffected, firefox101 wontfix, firefox102 fixed)
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox100 | --- | unaffected |
firefox101 | --- | wontfix |
firefox102 | --- | fixed |
People
(Reporter: itiel_yn8, Assigned: nchevobbe)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR:
- Using latest Nightly on Windows 10, fresh profile, enable the toolbox
- Open many tabs so that the tabbar would overflow and the "List all tabs" button would appear
- Open the toolbox, select Disable Popup Auto-Hide
- Click the "List all tabs" button on the tabbar
- Move back to the toolbox
AR:
- Sometimes, as soon as you move the cursor back to the toolbox window, the All Tabs list popup is closed but the List All Tabs button is in the mousedown'd state. Clicking it doesn't do anything and you must restart the browser to fix it.
- Other times, you can inspect that panel, but as soon as you close the toolbox, the same thing happens- the All Tabs list popup is closed but the List All Tabs button is in the mousedown'd state. Clicking it doesn't do anything and you must restart the browser to fix it.
ER:
I should be able to inspect that panel and that panel should work after closing the toolbox.
Comment 3•3 years ago
|
||
Set release status flags based on info from the regressing bug 1442600
Comment 4•3 years ago
|
||
:nchevobbe, since you are the author of the regressor, bug 1442600, could you take a look?
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 5•3 years ago
|
||
So I can confirm the issue with Bug 1442600 causing the tooltip to be hidden, I'll look into that.
Note that even when reverting the patch, with "Disable Popup Auto-Hide", the all tabs popup can't be hidden by clicking on the button again (Re enabling Popup Auto-Hide makes it possible to close it).
Assignee | ||
Comment 6•3 years ago
|
||
I think patch from Bug 1442600 is only revealing an issue that was already existing but that we'd never encounter
Here the popup is hidden when we show the highlighter, which we never did before because of the extra check in isXUL
function
Updated•3 years ago
|
Assignee | ||
Comment 7•3 years ago
|
||
I'm not a 100% sure, but I suspect that using insertAnonymousContent
on a document where we have a XUL popup displayed causes it to disappear.
Emilio, is this a known issue/something expected?
I'll go ahead and fix the specific issue we're seeing, but it might be good to sort this out
Assignee | ||
Comment 8•3 years ago
|
||
Using insertAnonymousContent on a document where a XUL popup is displayed seems
to hide/destroy it.
This patch extends the current behaviour for XUL windows to Chrome windows:
we add an iframe in the document which will contain the anonymous content.
Updated•3 years ago
|
Comment 9•3 years ago
|
||
insertAnonymousContent
for the first time in a document causes frame reconstruction of the whole document (somewhat expectedly).
That in turn is expected to close XUL popups. It should be feasible I think to implement more fine-grained invalidation of the anonymous content in particular in the frame constructor, though haven't looked at how easy or hard it'd be.
Assignee | ||
Comment 10•3 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #9)
insertAnonymousContent
for the first time in a document causes frame reconstruction of the whole document (somewhat expectedly).That in turn is expected to close XUL popups. It should be feasible I think to implement more fine-grained invalidation of the anonymous content in particular in the frame constructor, though haven't looked at how easy or hard it'd be.
Oh okay, thanks for the explanation.
I filed Bug 1768896 if we ever want to try to address this issue, and I'll add it as a reference in the patch for this bug.
I was wondering if I could workaround this another way, trying to retrieve the opened popups and reopen them after calling insertAnonymousContent
, but I'm afraid this would trigger unexpected events/behavior :/
Comment 11•3 years ago
|
||
Comment 12•3 years ago
|
||
bugherder |
Comment 13•3 years ago
|
||
The patch landed in nightly and beta is affected.
:nchevobbe, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 14•3 years ago
|
||
(In reply to Release mgmt bot [:suhaib / :marco/ :calixte] from comment #13)
The patch landed in nightly and beta is affected.
:nchevobbe, is this bug important enough to require an uplift?
If not please setstatus_beta
towontfix
.For more information, please visit auto_nag documentation.
Browser Toolbox only issue and the fix didn't include automated test, so I'd rather not uplift it
Description
•