Bug 56687 - Splash Screen Display Problems With 100 Or More Files
Summary: Splash Screen Display Problems With 100 Or More Files
Status: RESOLVED FIXED
Alias: None
Product: juk
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Scott Wheeler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-04-01 07:44 UTC by davidsmind
Modified: 2003-04-02 04:08 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
A picture of said bug (263.78 KB, image/png)
2003-04-02 03:21 UTC, davidsmind
Details

Note You need to log in before you can comment on or make changes to this 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!