Toolbar/tabs drag & drop doesn't work - Firefox snap wayland
Categories
(Core :: Widget: Gtk, defect, P3)
Tracking
()
People
(Reporter: jeremie.tamburini, Unassigned)
References
(Blocks 2 open bugs)
Details
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:98.0) Gecko/20100101 Firefox/98.0
Steps to reproduce:
Fresh installed Ubuntu 22.04 (still development version) with Firefox snap version 98.0.1
(Also tried version 99 and 100 available on https://snapcraft.io/firefox same behavior).
- Right click on toolbar
- Select "Customise Toolbar"
- Tried to remove items from the toolbar and add items on it via "drag and drop".
Actual results:
Nothing happens. It looks like drag and drop doesn't work.
While trying to move items, they simply disappear.
You can see a short video: https://www.youtube.com/watch?v=vsnr8O5EgFk
(Also tried to run Firefox from the terminal, but I didn't receive any warning while trying to customize the toolbar).
Expected results:
What should have happened is that every items should be moved between the toolbar and the list of items as shown in the official documentation:
https://support.mozilla.org/en-US/kb/customize-firefox-controls-buttons-and-toolbars
Comment 1•4 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Firefox::Toolbars and Customization' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 2•4 years ago
|
||
I've realized 2 new things:
- this problem is related to the Wayland session
- drag and drop doesn't work also with tabs
No problem at all selecting the xorg session at login, and this actually works as workaround.
I have seen other bug report mentioning similar problems. So my bug report could be a duplicated of 1739981 and 1739970
I'm going to update the first post.
Reporter | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 3•4 years ago
|
||
Can you try plain Mozilla nightly binary?
https://fedoraproject.org/wiki/How_to_debug_Firefox_problems?rd=Bug_info_Firefox#Testing_Mozilla_binaries
Thanks.
Reporter | ||
Comment 4•4 years ago
|
||
Thanks @Martin Stránský
I've run the test with Firefox nightly using the MOZ_ENABLE_WAYLAND=1 ./firefox -ProfileManager -no-remote
parameters and everything works as expected.
The problem seems to be confined to snap version of Firefox in Wayland session (snap nightly version included). The snap package works fine with the old X11 session.
Updated•4 years ago
|
Comment 7•3 years ago
|
||
Update from the launchpad bug:
I bisected mutter and gnome-shell until I identified the revision in mutter that caused the regression: https://gitlab.gnome.org/GNOME/mutter/-/commit/26676a829e74859488154cd8c45de1d0b629f3ca.
More specifically, the changes to src/core/events.c. Indeed I rebuilt mutter in jammy with the following patch, and the issue with the firefox snap was gone:
--- a/src/core/events.c
+++ b/src/core/events.c
@@ -523,10 +523,6 @@ meta_display_handle_event (MetaDisplay
#ifdef HAVE_WAYLAND
if (wayland_compositor && !bypass_wayland)
{
- if (window && event->type == CLUTTER_MOTION &&
- event->any.time != CLUTTER_CURRENT_TIME)
- meta_window_check_alive_on_event (window, event->any.time);
-
if (meta_wayland_compositor_handle_event (wayland_compositor, event))
bypass_clutter = TRUE;
}
(In reply to Olivier Tilloy from comment #7)
Update from the launchpad bug:
I bisected mutter and gnome-shell until I identified the revision in mutter that caused the regression: https://gitlab.gnome.org/GNOME/mutter/-/commit/26676a829e74859488154cd8c45de1d0b629f3ca.
More specifically, the changes to src/core/events.c. Indeed I rebuilt mutter in jammy with the following patch, and the issue with the firefox snap was gone:
--- a/src/core/events.c +++ b/src/core/events.c @@ -523,10 +523,6 @@ meta_display_handle_event (MetaDisplay #ifdef HAVE_WAYLAND if (wayland_compositor && !bypass_wayland) { - if (window && event->type == CLUTTER_MOTION && - event->any.time != CLUTTER_CURRENT_TIME) - meta_window_check_alive_on_event (window, event->any.time); - if (meta_wayland_compositor_handle_event (wayland_compositor, event)) bypass_clutter = TRUE; }
Do you have more context on this fix? I could use some assistance on how to implement it.
Updated•3 years ago
|
Comment 11•3 years ago
|
||
(In reply to jaz.zimms from comment #8)
Do you have more context on this fix? I could use some assistance on how to implement it.
There is nothing to implement: the issue is in gtk/mutter, and will be worked around/fixed there. See https://gitlab.gnome.org/GNOME/mutter/-/issues/2216 for details.
Comment 12•3 years ago
|
||
After testing locally, it looks like on a 22.04, nightly 103 snap (thus running wayland) I cannot repro. The upstream issue is still open, but maybe a workaround was landed in the distro. Do you still hit the issue ?
Comment 13•3 years ago
|
||
Also wondering if you still hit it, since your comment suggest you did?
Comment 14•3 years ago
|
||
Olivier, was a workaround landed and we could mark this as resolved?
Comment 15•3 years ago
|
||
On my Ubuntu 22.10 problem is resolved for bot FF 101.0.1 snap and firefox-trunk: Installed: 103.0a1hg20220615r620926-0ubuntu0.22.10.1~umd1
Comment 16•3 years ago
|
||
Yes, a workaround was distro-patched into mutter 42.0-3ubuntu1 in Ubuntu 22.04 (see https://launchpad.net/bugs/1964541 for details).
So I think it is safe to mark this resolved.
Updated•3 years ago
|
Updated•7 months ago
|
Description
•