After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 758609 - Regression with firefox dropdown menu position
Regression with firefox dropdown menu position
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Backend: X11
3.19.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2015-11-24 17:11 UTC by Olivier Fourdan
Modified: 2015-11-30 13:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gtkwindow: fix regression with firefox dropdown menu (1.34 KB, patch)
2015-11-26 10:05 UTC, Olivier Fourdan
none Details | Review
gtkwindow: fix regression with firefox dropdown menu (925 bytes, patch)
2015-11-26 12:30 UTC, Olivier Fourdan
accepted-commit_now Details | Review

Description Olivier Fourdan 2015-11-24 17:11:56 UTC
Summary:

gtk+ 3.19.x introduced a regression with firefox dropdown menu positionning.

How reproducible:

100%

Steps to reproduce:

1. Run firefox and tart typing a url so that the url dropdown menu appears.
2. Move firefox to another location
3. Type in the url bar again

Actual result:

The dropdown menu appears where it wa before firefox was moved

Expected result:

The dropdown menu appears at the correct location matching where the window is now.

Additional data:

A git bisect came out with commit 6866d1c as the first bad commit.
Comment 1 Olivier Fourdan 2015-11-25 08:45:38 UTC
There are actually two regressions.

1. The one described in comment 0 

This is caused by commit 6866d1c
Can be fixed with

--- a/gtk/gtkwindow.c
+++ b/gtk/gtkwindow.c
@@ -5565,6 +5565,7 @@ gtk_window_move (GtkWindow *window,
   else
     {
       /* Save this position to apply on mapping */
+      gtk_widget_queue_resize (widget);
       info->initial_x = x;
       info->initial_y = y;
       info->initial_pos_set = TRUE;

as suggested by Company on irc.

2. A regression that shows the dropdown menu in (0,0)

   Step to reproduce

   1. Run firefox and start typing a url so that the url dropdown menu appears.
   2. Dismiss the menu with "Escape"
   3. Type in the url bar again

   Actual result

   The dropdown menu now shows in (0,0)

A new git bisect gives commit 1f01b8d as the first bad commit.
Comment 2 Olivier Fourdan 2015-11-26 10:05:16 UTC
Created attachment 316293 [details] [review]
gtkwindow: fix regression with firefox dropdown menu

This is a patch based on a couple of fixes Company (Benjamin) posted on irc that fix the issue reported here.
Comment 3 Olivier Fourdan 2015-11-26 12:30:12 UTC
Created attachment 316310 [details] [review]
gtkwindow: fix regression with firefox dropdown menu

Updated patch as Benjamin already pushed a part of this patch.
Comment 4 Matthias Clasen 2015-11-30 13:16:26 UTC
Review of attachment 316310 [details] [review]:

ok
Comment 5 Olivier Fourdan 2015-11-30 13:34:54 UTC
attachment  pushed a commit f30b4ba gtkwindow: fix regression with firefox dropdown menu