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 379742 - Broken first time sidebar resize
Broken first time sidebar resize
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Movie player
2.17.x
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
: 383021 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-11-27 10:50 UTC by Bastien Nocera
Modified: 2007-02-23 11:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Bastien Nocera 2006-11-27 10:50:30 UTC 
1. Start Totem
2. Hide the sidebar
3. Exit Totem
4. Start Totem
5. Show the sidebar

At 5., the window will not be resized for the sidebar, but use the same window width. It works as expected after the first show.
Comment 1 Bastien Nocera 2006-11-27 11:00:28 UTC 
In cb_resize:
        g_message ("alloc %d", totem->sidebar->allocation.width);
gives out:
** Message: alloc 1

This is obviously not right.
Comment 2 Bastien Nocera 2006-12-06 15:09:48 UTC 
*** Bug 383021 has been marked as a duplicate of this bug. ***
Comment 3 Eric Anderson 2007-02-23 03:21:12 UTC 
The sidebare is not being realized. I guess someone removed it because they thought it wasn't needed. It needs to be realized, otherwise we think it takes up 0 space.

It just needs something like the following in totem-sidebar-setup.

  gtk_widget_show_all (totem->sidebar);

+ gtk_widget_realize(totem->sidebar);

  if (!visible)
 	gtk_widget_hide (totem->sidebar);

I didn't see too much sense in attaching a patch for that.
Comment 4 Eric Anderson 2007-02-23 03:24:11 UTC 
It was a regression caused here:
http://svn.gnome.org/viewcvs/totem/trunk/src/totem-sidebar.c?r1=3703&r2=3713
Comment 5 Bastien Nocera 2007-02-23 11:14:34 UTC 
We still suffer from bug #349937 though.

2007-02-23  Bastien Nocera  <hadess@hadess.net>

        * src/totem-playlist.c: (totem_playlist_init),
        (totem_playlist_new): Move _new stuff to _init
        * src/totem-sidebar.c: (totem_sidebar_setup): Fix first-time sidebar
        resize, as pointed out by Eric Anderson <ejona86@gmail.com>
        (Closes: #379742)