[Nightly + macOS-only] With native context menus enabled the drag image blinks when clicking "Reload Tab" menu, if using Ctrl-click to open the tab context menu
Categories
(Core :: Widget: Cocoa, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox-esr91 | --- | disabled |
firefox90 | --- | disabled |
firefox91 | --- | disabled |
firefox92 | --- | fixed |
People
(Reporter: suishouen, Assigned: mstange)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression, Whiteboard: [mac:mr1][mac:integration])
Attachments
(2 files)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:89.0) Gecko/20100101 Firefox/89.0
Steps to reproduce:
- Set "browser.proton.enabled to true" and "widget.macos.native-context-menus to true"
- Launch Nightly
- Open any page (e.g. https://www.mozilla.org/en-US/firefox/)
- Open tab menu (control click the tab)
- Click "Reload Tab"
Actual results:
The drag image blinks.
Expected results:
The drag image should not blink.
Comment 1•4 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Firefox::Menus' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.
Comment 2•4 years ago
|
||
Hi Eiichi,
I set up the config settings you mentioned but I am not particularly sure if I can't reproduce the issue or I am not looking at what I should.
Could you make a screen recording when you reproduce it and share it here?
Thanks!
For reference;
When I use Nightly, I always rename 'Firefox Nightly' to 'Namoroka'.
So the name of Application shows 'Namoroka' in the Screen Recording Movie.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
When setting "widget.macos.native-context-menus" to "false", this issue doesn't happen.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 6•4 years ago
|
||
This only happens after bug 1615732; turning off dom.event.treat_ctrl_click_as_right_click.disabled
makes this problem go away. That pref is only set to true by default on nightly, so we won't have this issue in 89 release.
Assignee | ||
Comment 7•4 years ago
|
||
This bug can also lead to accidental tab detaches: Ctrl-click on the tab, and then dismiss the menu by doing a small drag gesture outside the menu over the content area. But, as Gijs said, only on Nightly.
Comment 8•4 years ago
|
||
Yeah, this looks like the Ctrl-click triggers a drag session, maybe we should prevent ctrl-click to start a drag session on Mac as it would bring up the context menu.
Assignee | ||
Comment 9•4 years ago
|
||
We have this piece of code to unset capturing content and :active state when the native menu opens, maybe we need something similar for aborting drag detection: https://searchfox.org/mozilla-central/rev/759872688df15a5d6ab305ffe39d90450590bfec/layout/xul/nsXULPopupManager.cpp#820-828
Updated•4 years ago
|
Comment 10•4 years ago
|
||
I now understand from comment 6 that this will not happen in release 89 given that dom.event.treat_ctrl_click_as_right_click.disabled won't be enabled on 89 but at some point later. Removing Proton tracking whiteboad items for this reason
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 11•4 years ago
|
||
I'm not sure if this is a good fix - basically, we want to cancel all effects from a mousedown.
A number of places call both ClearGlobalActiveContent and ReleaseCapturingContent. It seems like
all those places would be interested in canceling drag tracking as well.
The caller I care about for this particular bug is nsXULPopupManager::ShowPopupAsNativeMenu.
Updated•4 years ago
|
Updated•3 years ago
|
Comment 14•3 years ago
|
||
Comment 15•3 years ago
|
||
bugherder |
Comment 16•3 years ago
|
||
Is this something we should consider for 91?
Assignee | ||
Comment 17•3 years ago
|
||
Probably not worth it - this only affects build with dom.event.treat_ctrl_click_as_right_click.disabled
set to true, and that pref is only true by default on Nightly, not on Beta/Release.
Updated•3 years ago
|
Description
•