diff options
| author | Patrick Storz <eduard.braun2@gmx.de> | 2019-02-19 23:40:10 +0000 |
|---|---|---|
| committer | Patrick Storz <eduard.braun2@gmx.de> | 2019-02-20 01:44:47 +0000 |
| commit | bbfd5e166a8adaf0e76919d5ee13e68db1f256c9 (patch) | |
| tree | 99ed10c824e05345977140022e3e6c02cc80d154 /src/object | |
| parent | Rescue code for restoring last used window geometry (diff) | |
| download | inkscape-bbfd5e166a8adaf0e76919d5ee13e68db1f256c9.tar.gz inkscape-bbfd5e166a8adaf0e76919d5ee13e68db1f256c9.zip | |
Fix query for monitor dimensions
The call to
Gdk::Display::get_monitor_at_window()
in
Inkscape::UI::get_monitor_geometry_at_window()
requires the underlying GdkWindow to be fully initialized.
This is achieved by calling "realize()" before attempting to read
that information.
Previously we used to show the window first (which implies realizing
it) which worked around the issue. However it required us to hide
and show it once again later if we wanted to move it in order for
the window manager's routines to sanitize the position to kick in.
Diffstat (limited to 'src/object')
| -rw-r--r-- | src/object/sp-namedview.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/object/sp-namedview.cpp b/src/object/sp-namedview.cpp index e4ab351d1..eafdf6dba 100644 --- a/src/object/sp-namedview.cpp +++ b/src/object/sp-namedview.cpp @@ -819,11 +819,7 @@ void sp_namedview_window_from_document(SPDesktop *desktop) #endif desktop->setWindowSize(w, h); if (move_to_screen) { - // Hiding window will close app if it's last window. If we really need to hide it - // here, we need to up the reference count of the application before hiding, and lower after showing. - // win->hide(); desktop->setWindowPosition(Geom::Point(nv->window_x, nv->window_y)); - // win->show(); } } } |