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 368757 - progress bar too big in statusbar
progress bar too big in statusbar
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: User Interface
0.9.x
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 323396 377572 460641 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-11-01 08:42 UTC by Baptiste Mille-Mathias
Modified: 2008-06-08 14:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
totem buffering progress bar (64.92 KB, image/png)
2006-11-01 08:43 UTC, Baptiste Mille-Mathias
  Details
Rb progressbar (911 bytes, image/png)
2006-11-01 08:52 UTC, Baptiste Mille-Mathias
  Details
a patch that correct the ui (2.05 KB, patch)
2006-11-12 10:08 UTC, Baptiste Mille-Mathias
none Details | Review
previous patch was wrong (7.65 KB, patch)
2006-11-12 10:21 UTC, Baptiste Mille-Mathias
needs-work Details | Review
Patch to copy the behaviour of totem (2.15 KB, patch)
2008-01-11 07:55 UTC, Sebastian Keller
none Details | Review
Also fix problem text-width (2.15 KB, patch)
2008-01-11 21:00 UTC, Sebastian Keller
none Details | Review

Description Baptiste Mille-Mathias 2006-11-01 08:42:52 UTC
Would be nice to use a mini progress bar as totem does.
screenshot to come

thanks
Comment 1 Baptiste Mille-Mathias 2006-11-01 08:43:58 UTC
Created attachment 75754 [details]
totem buffering progress bar
Comment 2 Baptiste Mille-Mathias 2006-11-01 08:52:45 UTC
Created attachment 75757 [details]
Rb progressbar

As seen, the top border of the progress bar overlaps the bottom border of the
rendering window.
Some padding between the progressbar would give a better finition, but as the
statusbar's height cannot be increased, reducing the height of the progressbar
as totem did would be nice.
Comment 3 Baptiste Mille-Mathias 2006-11-01 10:24:09 UTC
I thought this patch would have worked, but the progressbar height doesn't change.

Index: shell/rb-statusbar.c
===================================================================
RCS file: /cvs/gnome/rhythmbox/shell/rb-statusbar.c,v
retrieving revision 1.51
diff -u -r1.51 rb-statusbar.c
--- shell/rb-statusbar.c        15 Aug 2006 09:42:59 -0000      1.51
+++ shell/rb-statusbar.c        1 Nov 2006 10:19:53 -0000
@@ -158,6 +158,7 @@
         statusbar->priv->loading_text = g_strdup (_("Loading..."));
 
         gtk_progress_bar_set_fraction (GTK_PROGRESS_BAR (statusbar->priv->progress), 1.0);
+               gtk_widget_set_size_request ((GTK_WIDGET (statusbar->priv->progress)), 150, 10);
         gtk_widget_hide (statusbar->priv->progress);
 
         gtk_box_pack_start (GTK_BOX (statusbar),
Comment 4 Baptiste Mille-Mathias 2006-11-12 10:08:06 UTC
Created attachment 76423 [details] [review]
a patch that correct the ui

This patch works fine.
Comment 5 Baptiste Mille-Mathias 2006-11-12 10:21:37 UTC
Created attachment 76424 [details] [review]
previous patch was wrong
Comment 6 James "Doc" Livingston 2006-11-12 11:48:09 UTC
Looks fine to me, committed to cvs.
Comment 7 Alex Lancaster 2006-11-21 10:02:24 UTC
*** Bug 377572 has been marked as a duplicate of this bug. ***
Comment 8 Alex Lancaster 2006-11-21 10:03:25 UTC
Re-opening because fix means that progress bar no longer fits text, so we need a better solution.
Comment 9 William Jon McCann 2006-12-07 17:03:53 UTC
This should be reverted due to the regression noted in bug 377572.
Comment 10 James "Doc" Livingston 2006-12-09 00:58:39 UTC
I've reverted the patch.
Comment 11 Sebastian Keller 2008-01-11 07:55:36 UTC
Created attachment 102576 [details] [review]
Patch to copy the behaviour of totem

I created a patch that copies the behaviour of totem and epiphany. This works with bigger fonts unlike the old patch. It fixes the jumping statusbar-problem caused by the progressbar being bigger than the normal label in the statusbar and causing the statusbar to "jump" everytime you switched the artist or the album. However it has a small problem with the statusbar getting drawn over the resize grip for a very short moment from time to time. Epiphany also has this problem and i think this is at least better than to have the whole statusbar jumping.
Comment 12 Jonathan Matthew 2008-01-11 10:51:47 UTC
That still doesn't fit the text properly.  While ripping a CD, all I see in the progress bar is 'nsferring track 6 out of 9 (1'.
Comment 13 Sebastian Keller 2008-01-11 21:00:33 UTC
Created attachment 102625 [details] [review]
Also fix problem text-width

This is an updated version of the patch to fix the width of the text, too.
Comment 14 Jonathan Matthew 2008-06-07 11:21:46 UTC
*** Bug 460641 has been marked as a duplicate of this bug. ***
Comment 15 Jonathan Matthew 2008-06-07 11:34:35 UTC
It seems that just setting the size request on the progress bar has more or less the desired effect, and doesn't clip text noticeably.
Comment 16 Jonathan Matthew 2008-06-07 11:35:48 UTC
*** Bug 323396 has been marked as a duplicate of this bug. ***
Comment 17 Jonathan Matthew 2008-06-08 14:53:56 UTC
I've committed a change that just sets the size request on the progress bar.  Hopefully this will go away now.