Closed Bug 1698377 Opened 4 years ago Closed 4 years ago

Profiler button dropdown still appears after the button overflows

Categories

(Firefox :: Toolbars and Customization, defect, P2)

defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox89 --- verified
firefox90 --- verified

People

(Reporter: itiel_yn8, Assigned: mhowell)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [proton-toolbar])

Attachments

(6 files)

Attached image Screenshot (deleted) —

See attached (I made both items with the hover effect just so the styling for both would be clear)

Mark, can you prio this for toolbar / hamburger menu work? Is this perhaps already covered under bug 1698436 ?

Severity: -- → S4
Flags: needinfo?(mstriemer)
Whiteboard: [proton-toolbar]
Priority: -- → P2

This is being handled in bug 1698436.

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(mstriemer)
Resolution: --- → DUPLICATE
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---

To be clear, the bug in comment 3 is that the dropdown appeared at all; it's only supposed to be there when the button is in the toolbar, not in the overflow list.

@alex_mayorga, do you have a reliable way to reproduce this? I've been trying and I can't seem to make it happen at all, nor can I see a way that it looks like it could (not one that's obvious enough for me to find it, anyway).

Flags: needinfo?(alex_mayorga)

¡Hola Molly!

All that I need to do is to reduce the window to half the screen either left or right so the profiler hides under the overflow menu >> , then click the overflow menu >> and the bug manifest itself.

Hope this helps.

¡Gracias!
Alex

Flags: needinfo?(alex_mayorga)
Attached image image.png (deleted) —
Attached image image.png (deleted) —

Ooooooh, it's when the button is in the toolbar but it overflows into that menu, okay, thank you, that's exactly what I needed to know, I think I know how to fix this now.

Assignee: nobody → mhowell
Status: REOPENED → ASSIGNED
Regressed by: 1650835
Summary: The Profiler button is styled incorrectly when in the overflow list, if Proton is enabled → Profiler button dropdown still appears after the button overflows
Has Regression Range: --- → yes

When a toolbar button overflows out of the toolbar and into the overflow menu,
CustomizableUI continues to report the widget's type as "toolbar". This means
that whether the widget is currently overflowed must be considered in addition
to the widget's type when the panel and toolbar states need different handling.

Pushed by mhowell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7032f1775c4b Treat button-and-view widgets as being in a panel when they overflow off the toolbar. r=Gijs
Status: ASSIGNED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Flags: qe-verify+
Attached image profiler overflow cut win 10.jpg (deleted) —

Reproduced the issue on Firefox (2021-03-14) under macOS 10.15.7. The issue seems to be fixed on Firefox 89.0b9 and 90.0a1 under macOS 10.15.7 and Ubuntu 20.04.

Windows 10 on the other hand still seems to have the profiler in overflow cut (see attachment). Anything we can do?

Flags: needinfo?(mhowell)

Oh, sorry, this bug wasn't for the panel itself overflowing, this was about the button that opens the panel. That should be a separate bug.

Flags: needinfo?(mhowell)

Understood, then the issue for the button that opens the panel is fixed as verified, as mentioned in Comment 12.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: