Rhythmbox's window does not remember its on-screen position when unhiding from status icon

[Alexander Jones]

Binary package hint: rhythmbox

Move RB to center of screen
Click RB icon in status icon area to hide the window
Click it again to unhide it

The window is placed at (0, 0), instead of wherever it was when it was hidden away.

ProblemType: Bug
Architecture: i386
Date: Sun Dec 9 20:52:00 2007
DistroRelease: Ubuntu 8.04
ExecutablePath: /usr/bin/rhythmbox
NonfreeKernelModules: vmnet vmblock vmmon fglrx
Package: rhythmbox 0.11.3-0ubuntu4
PackageArchitecture: i386
ProcCmdline: rhythmbox
ProcCwd: /home/alex
ProcEnviron:
 SHELL=/bin/bash
 PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=en_GB.UTF-8
 LANGUAGE=en_GB:en
SourcePackage: rhythmbox
Uname: Linux flash 2.6.22-14-generic #1 SMP Sun Oct 14 23:05:12 GMT 2007 i686 GNU/Linux

[Pedro Villavicencio (pedro)]

Thanks for your bug report. This bug has been reported to the developers of the software. You can track it and make comments here: http://bugzilla.gnome.org/show_bug.cgi?id=502827

[pelle.k (pele2)]

Hey. I've attached a patch i made for rhythmbox 0.11.6svn20081008-0ubuntu3 in intrepid (ubuntu 8.10).
It's very light, only one line adjusted in shell/rb-shell.c

Bug description;
The method rb_shell_set_visibility saves the window state even after the window has been hidden, and thus sets window_x and window_y values to zero.

With this patch applied, rhythmbox will also start from the last position it was minimized from (since window position is saved in gconf).

[pelle.k (pele2)]

Sorry to bump this bug, but i've attached a new patch in the bugzilla bugtracker. I assume it's best if the discussion continue there. If you want to try the patched binary for intrepid, or get the source package;
http://download.tuxfamily.org/pandora/temp/rhytmbox-intrepid/
Would be nice if we could get it into intrepid though, so please try it out and confirm it's working.

[Pedro Villavicencio (pedro)]

this has been fixed upstream thanks for reporting.

[Sebastien Bacher (seb128)]

This update fixes the issue

 rhythmbox (0.11.6svn20090108-0ubuntu1) jaunty; urgency=low
 .
   * New svn update
   * debian/control.in:
     - updated gtk requirement
   * debian/patches/90_correct_cd_recording.patch,
     debian/patches/90_upstream_change_correct_cd_copy_naming.patch,
     debian/patches/91_upstream_fix_mtp_transfer.patch,
     debian/patches/92_upstream_change_correct_player_listing.patch,
     debian/patches/93_upstream_change_no_crossfading_locking.patch:
     - the changes are in the new version
   * debian/patches/01_lpi.patch:
     - new version update
   * debian/patches/02_autoconf.patch:
     - new version update