Bug 130564

Summary:	multimeter isn't viewable with tiny kicker
Product:	[Unmaintained] ksysguard	Reporter:	Andrew Berry <andrewberry>
Component:	general	Assignee:	KSysGuard Developers <ksysguard-bugs>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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 :).