2015-06-15 (CEST time) GIMPnet #gnome
17:27:34 < Lekensteyn> In GNOME3, gtk_window_move(window, 0, 0) puts the window behind the top bar. Is it possible to make this window appear below the bar?
17:31:06 < borschty> Lekensteyn, do you use gnome-shell?
17:33:24 < Lekensteyn> borschty: actually, the widnow is not hidden behind the top bar, but the top bar of the window just dissappears
17:33:33 < Lekensteyn> see the screenshots in https://bugs.archlinux.org/task/45274#comment136419
17:33:55 < Lekensteyn> (I am not a user, don't know whether this is gnome-shell or something else)
17:35:08 < borschty> maybe the user is using some gnome-shell extension to hide the window border for maximized windows?
17:36:00 < Lekensteyn> no it is not, it is Wireshark-specific code that tries to remember the position
17:36:11 < Lekensteyn> it has worked on older gnome versions, kde, etc
17:36:29 < Lekensteyn> but it broke for gnome3, so now I'm trying to fix that
17:36:49 < Lekensteyn> not sure if it is a gnome3 bug or a programming error
17:37:13 < Lekensteyn> but by requesting movement to (0, 0) I would expect the upper left corner to be aligned to the screen
17:37:44 < borschty> gnome-shell automatically maximizes windows that are closed to maximized size to work around applications not storing the maximized state correctly
17:38:33 < borschty> it should still show the titlebar though
17:39:20 < bjsnider> that's multiple monitor, maybe that has something to do with it
17:39:28 < bjsnider> does it happen on the other monitor too?
17:39:36 < Lekensteyn> no it does not afaik
17:40:02 < Lekensteyn> the steps to reproduce are described at the comment before the pictures
17:40:05 < borschty> and could you try this with a smaller window size
17:40:19 < Lekensteyn> with a smaller window size it does not happen
17:40:38 < Lekensteyn> so with a display size of 900px, requesting a height of 900 exposes this issue
17:40:43 < Lekensteyn> but using 800px instead works fine
17:40:55 < bjsnider> but does it happen with a gtk app other than wireshark
17:41:04 < borschty> maybe wireshark tries to create window that is too large
17:41:28 < borschty> yeah, the window size does not include the titlebar
17:41:48 < Lekensteyn> (for a context, I'm a Wireshark developer trying to resolve this issue)
17:41:52 < borschty> so if it requests a 1600px window on a 1600px screen, there is no room for a titlebar
17:42:26 < Lekensteyn> but how should the application get knowledge about the titlebar?
17:42:35 < Lekensteyn> the height, that is
17:42:47 < Lekensteyn> it is a property of the window manager (mutter)
17:43:12 < Lekensteyn> the desire is to set the window size, not the size of the contents
17:44:08 < Lekensteyn> I'll try to reproduce it with gtk2
17:46:39 < borschty> there should be no need to query the size of the titlebar, the window size should already be adjusted for this, so either wireshark is doing something wrong getting the window size to store in that file or there is some bug in gtk causing the reported window size to be wrong
17:51:08 < Lekensteyn> the height (1024 for a 1280x1024 display) is passed straight to gtk_window_resize(window, width, height)
17:59:25 < Lekensteyn> borschty: so the resize functionality is implemented here: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=blob;f=ui/gtk/gui_utils.c;h=c85d347825a34eb5a43dade6b62003e87018bd4c;hb=HEAD#l420
17:59:39 < Lekensteyn> seems pretty standard to me
18:02:36 < borschty> i was thinking that the problem might be more in the part that stores the size
18:03:35 < Lekensteyn> that value is taken straight from the configuration file. Just to be sure, it's currently ran through a debugger while a gtk2 build is being prepared
18:03:48 < Lekensteyn> Working on it with Balint at Sharkfest :-)
18:05:43 < borschty> yes, but if it uses the full screen height in the config file already, the problem is how that value got in there
18:07:14 < Lekensteyn> dk_window_get_height(widget_window) is used
18:07:20 < Lekensteyn> gdk_window_get_height(widget_window) that is*
18:07:41 < Lekensteyn> code is at https://code.wireshark.org/review/gitweb?p=wireshark.git;a=blob;f=ui/gtk/gui_utils.c;h=c85d347825a34eb5a43dade6b62003e87018bd4c;hb=HEAD#l341
18:09:15 < Lekensteyn> The problem is basically in the interpretation by gtk3 (or GNOME3?). As a developer I meant to set the window size, but the library assumes the contents size instead
18:09:19 < borschty> try using https://developer.gnome.org/gtk3/stable/GtkWindow.html#gtk-window-get-size
18:11:11 < borschty> the problem is that wireshark seems to be storing the window size including the decoration and then uses that to set the size of the window content
18:11:38 < Lekensteyn> it seems like that function would work in combination with the setter function, but people with older configuration files still have the issue
18:13:00 < borschty> in that case you could file a bug report against mutter about it not reducing the window size for windows on second screens if they don't fit the screen
18:14:15 < Lekensteyn> ah okay, was not sure about the exact component. Thank you for your assistance!