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 313893 - Source List resized automagically when create / delete playlist
Source List resized automagically when create / delete playlist
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: User Interface
HEAD
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-08-18 21:14 UTC by Baptiste Mille-Mathias
Modified: 2005-09-24 09:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (2.16 KB, patch)
2005-09-19 15:50 UTC, James "Doc" Livingston
none Details | Review
better patch (2.62 KB, patch)
2005-09-24 07:51 UTC, James "Doc" Livingston
committed Details | Review

Description Baptiste Mille-Mathias 2005-08-18 21:14:59 UTC
With a CVS version of RB (18-Aug-05):

1/ Be sure the Source List is displayed
2/ Create a new playlist with shortcut 

Result:
the Source List is resized
The same behaviour appears when deleting playlist

I think the problem appears after patch from bug 161935 was commited to CVS.
Comment 1 James "Doc" Livingston 2005-08-19 06:58:12 UTC
I think I've found the cause of the problem:

a) the patch in bug 161935 forces RB to come out of small mode when a playlist
is created.
b) when RB gets told to come out of small mode it maximises the window, even is
it is not in small mode.


I think the answer if to either a) don't maximise when coming out of small mode
(return to the previous size) or b) ignore the request to come out of small mode
if we are were not in it previously.
Comment 2 James "Doc" Livingston 2005-09-19 15:16:39 UTC
I've found the actual problem: the gconf key CONF_STATE_WINDOW_MAXIMIZED is
used, but isn't actually set anywhere. Fixing this should just be a matter of
setting this when the window is maximised/unmaximised.
Comment 3 James "Doc" Livingston 2005-09-19 15:50:52 UTC
Created attachment 52393 [details] [review]
patch

The patch makes Rhythmbox actually stop the maximisation state, window width
and height in the gconf keys. It should also fix bugs 137068 and 310252.
Comment 4 James "Doc" Livingston 2005-09-24 07:51:48 UTC
Created attachment 52582 [details] [review]
better patch

This patch actually fixes the first of the two other bug I mentioned, but still
doesn't fix the latter though. We should also probably monitor the gconf keys
for changes, but that will probably lead to issues (e.g. when the state is
changed quickly) because of the asynchonous nature of gconf.
Comment 5 James "Doc" Livingston 2005-09-24 09:24:27 UTC
Patch committed.