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 64396 - Window position not correctly stored on exit
Window position not correctly stored on exit
Status: RESOLVED FIXED
Product: galeon
Classification: Deprecated
Component: User interface
0.x
Other Linux
: Normal trivial
: ---
Assigned To: Daniel Erat
Daniel Erat
Depends on:
Blocks:
 
 
Reported: 2001-11-12 18:53 UTC by Torsten Hilbrich
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Torsten Hilbrich 2001-11-12 18:53:48 UTC
Galeon seems to not correctly store the window position on exit.

For example, the current galeon window has the following properties
(shorted to the relevant information):

xwininfo: Window id: 0xa00416 "Enter Bug - Galeon"

  Absolute upper-left X:  56
  Absolute upper-left Y:  16
  Relative upper-left X:  4
  Relative upper-left Y:  16
  Width: 910
  Height: 693
  ...
  Corners:  +56+16  -58+16  -58-59  +56-59
  -geometry 910x693+52+0

If I close the galeon right now it stores the upper left corner of
(56,16) into the ~/.g.galeon/session_saved.xml file instead of the
correct (52,0) as shown in the geometry argument.

If I start galeon next time it restore its position to (56,16) instead
of (52,0), shifting the window a bit to the lower right.  If I repeat
this step the window is shifted to the lower right with each start.

It would be nice if this (probably trivial) bug could be fixed.

Thanks,

             Torsten
Comment 1 alfredo 2001-11-12 20:28:03 UTC
There is a call to gdk_window_get_position in the line 313 of
src/session.c
Perhaps it should be a call to gdk_window_get_root_origin?
Comment 2 Marco Pesenti Gritti 2001-11-15 11:05:58 UTC
Dan, this is your code ?
Comment 3 Daniel Erat 2001-11-15 18:27:16 UTC
Nope, not my code, but I'm seeing the bug too, and will fix it later
today in both branches (I currently have a bunch of other unrelated
changes in my source tree that I need to test more before committing).
Comment 4 Daniel Erat 2001-11-16 07:29:45 UTC
Yup, gdk_window_get_root_origin is what was needed.  It's fixed now.