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 693781 - Jumping settings dialog
Jumping settings dialog
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Network
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
3.8.1?
Depends on:
Blocks:
 
 
Reported: 2013-02-14 10:38 UTC by Allan Day
Modified: 2013-04-16 09:46 UTC
See Also:
GNOME target: ---
GNOME version: 3.7/3.8


Attachments
network: Avoid jumping dialogs (1.39 KB, patch)
2013-04-13 01:18 UTC, Matthias Clasen
needs-work Details | Review
network: Avoid jumping dialogs (2.40 KB, patch)
2013-04-15 12:51 UTC, Matthias Clasen
committed Details | Review

Description Allan Day 2013-02-14 10:38:05 UTC
When I open the settings dialog for my wi-fi network, the dialog opens in an incorrect position (offset bottom-right from the main control center window) and then jumps to the correct location over the main window.

Dialogs should open in the correct position and not jump around.
Comment 1 Matthias Clasen 2013-04-13 01:18:39 UTC
Created attachment 241430 [details] [review]
network: Avoid jumping dialogs

The connection editor window is shown before all of its pages are
initialized, causing it to jump when it gets its final size. Avoid
that by not showing it before it is ready.
Comment 2 Bastien Nocera 2013-04-15 08:40:23 UTC
Review of attachment 241430 [details] [review]:

::: panels/network/connection-editor/net-connection-editor.c
@@ +938,3 @@
 net_connection_editor_present (NetConnectionEditor *editor)
 {
+        g_idle_add ((GSourceFunc)present_cb, editor);

We already have a g_idle_add() validating the pages' configurations (which I'd rather not have either).

You could show the window in recheck_initialization() instead.
Comment 3 Matthias Clasen 2013-04-15 12:51:06 UTC
Created attachment 241561 [details] [review]
network: Avoid jumping dialogs

The connection editor window is shown before all of its pages are
initialized, causing it to jump when it gets its final size. Avoid
that by not showing it before it is ready.
Comment 4 Bastien Nocera 2013-04-16 09:46:44 UTC
I've removed the unnecessary changes in the idle callback. Thanks for the patch!

Attachment 241561 [details] pushed as eb9c61f - network: Avoid jumping dialogs