Bug 130564 - multimeter isn't viewable with tiny kicker
Summary: multimeter isn't viewable with tiny kicker
Status: RESOLVED FIXED
Alias: None
Product: ksysguard
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-10 19:28 UTC by Andrew Berry
Modified: 2006-12-04 05:40 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Berry 2006-07-10 19:28:43 UTC
Version:            (using KDE KDE 3.5.2)
Installed from:    Ubuntu Packages

If the kicker size is set to "tiny", the multimeter view for ksysguard sensors isn't viewable. As soon as the kicker size is set to "small" or larger, the numbers in the sensors become viewable.

Thanks,
--Andrew
Comment 1 John Tapsell 2006-12-04 04:16:02 UTC
SVN commit 610290 by johnflux:

QLCDWidget seems to have a bug that it won't draw when the height is below a certain amount.

However there was also a bug/feature that it drew a white line at the bottom.  By using this extra pixel it is just tall enough to display the numbers even on a tiny 
kicker. :-)

BUG: 130564:


 M  +2 -2      MultiMeter.cc  


--- branches/KDE/3.5/kdebase/ksysguard/gui/SensorDisplayLib/MultiMeter.cc #610289:610290
@@ -63,7 +63,7 @@
 
 	setPlotterWidget(lcd);
 
-	setMinimumSize(16, 16);
+	setMinimumSize(5, 5);
 	setModified(false);
 }
 
@@ -134,7 +134,7 @@
 MultiMeter::resizeEvent(QResizeEvent*)
 {
 	if (noFrame())
-		lcd->setGeometry(0, 0, width() - 1, height() - 1);
+		lcd->setGeometry(0, 0, width(), height());
 	else
 		frame()->setGeometry(0, 0, width(), height());
 }
Comment 2 John Tapsell 2006-12-04 04:17:23 UTC
SVN commit 610291 by johnflux:

Give QLCDWidget enough room to draw on a tiny kicker
CCBUG: 130564


 M  +2 -2      MultiMeter.cc  


--- trunk/KDE/kdebase/workspace/ksysguard/gui/SensorDisplayLib/MultiMeter.cc #610290:610291
@@ -57,7 +57,7 @@
 
   setPlotterWidget( mLcd );
 
-  setMinimumSize( 16, 16 );
+  setMinimumSize( 5, 5 );
 }
 
 bool MultiMeter::addSensor(const QString& hostName, const QString& sensorName,
@@ -115,7 +115,7 @@
 
 void MultiMeter::resizeEvent( QResizeEvent *event )
 {
-  mLcd->setGeometry(0, 0, width() - 1, height() - 1);
+  mLcd->setGeometry(0, 0, width(), height());
 
   QWidget::resizeEvent( event );
 }
Comment 3 Andrew Berry 2006-12-04 05:40:17 UTC
TYVM :).