Bug 56687

Summary:	Splash Screen Display Problems With 100 Or More Files
Product:	[Applications] juk	Reporter:	davidsmind
Component:	general	Assignee:	Scott Wheeler <wheeler>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	A picture of said bug

Description davidsmind 2003-04-01 07:44:25 UTC

Version:            (using KDE 3.1.9)
Compiler:          gcc version 3.2.2
OS:          Linux (i686) release 2.4.21-pre5-aa2

When the splash screen indicates more then one hundred files loded it says "Items Loade 602" When it should actually expand and show "Items Loaded 602" but the splash screen size remains the same and the number obscures the text.

Comment 1 Scott Wheeler 2003-04-02 01:15:04 UTC

Subject: Re: Splash Screen Display Problems With 100 Or More Files

Hmm.  I can't seem to get that here.  Could you possibly send a screen shot?

Comment 2 Stephen Douglas 2003-04-02 01:19:48 UTC

I can't get it either, and I have 2640-odd files - are you using an unusual font?

Comment 3 davidsmind 2003-04-02 03:21:03 UTC

Created attachment 1287 [details]
 A picture of said bug

My Font is Trubuchet MS, a True Type font I believe.

Comment 4 Scott Wheeler 2003-04-02 03:31:16 UTC

Subject: kdemultimedia/juk

CVS commit by wheeler: 

Heh, found it!  I had all of the logic in there to ensure that the label
was wide enough, but sadly was applying it to the wrong label.  ;-)

CCMAIL:56687-done@bugs.kde.org


  M +1 -1      splashscreen.cpp   1.8


--- kdemultimedia/juk/splashscreen.cpp  #1.7:1.8
@@ -83,9 +83,9 @@ SplashScreen::SplashScreen() : QHBox(0 ,
     QLabel *textLabel = new QLabel(i18n("Items loaded:"), this);
     textLabel->setFont(font);
-    textLabel->setMinimumWidth(textLabel->fontMetrics().width("00000"));
 
     m_countLabel = new QLabel(this);
     m_countLabel->setText(QString::number(count));
     m_countLabel->setFont(font);
+    m_countLabel->setMinimumWidth(m_countLabel->fontMetrics().width("00000"));
     
     setMaximumWidth(iconLabel->width() + textLabel->width() + m_countLabel->width() + 10);

Comment 5 Scott Wheeler 2003-04-02 04:04:51 UTC

Subject: Re:

Ok, I just closed this because I'm pretty sure I spotted it and fixed it.  If 
you're running HEAD can you give it a try?  Otherwise I can send you a patch 
againt 1.1.

Comment 6 davidsmind 2003-04-02 04:08:54 UTC

Thank you for the prompt bug-busting!