- fix and rewrite draw algorithm to work with larger sizes

- add alternative appearance (hint: click'n'hold on the battery meter :-)

2 files changed, 55 insertions, 40 deletions

diff --git a/core/applets/batteryapplet/battery.cpp b/core/applets/batteryapplet/battery.cpp
index f3a95ed..eee3ed3 100644
--- a/core/applets/batteryapplet/battery.cpp
+++ b/core/applets/batteryapplet/battery.cpp
@@ -21,6 +21,8 @@
 #include "batterystatus.h"
 
-#include <qpe/power.h>
 #include <qpe/applnk.h>
+#include <qpe/config.h>
+#include <qpe/power.h>
+#include <qpe/qpeapplication.h>
 
 #include <qpainter.h>
@@ -37,4 +39,8 @@ BatteryMeter::BatteryMeter( QWidget *parent )
     connect( chargeTimer, SIGNAL(timeout()), this, SLOT(chargeTimeout()) );
     timerEvent(0);
+    QPEApplication::setStylusOperation( this, QPEApplication::RightOnHold );
+    Config c( "qpe" );
+    c.setGroup( "Battery" );
+    style = c.readNumEntry( "Style", 0 );
 }
 
@@ -49,11 +55,23 @@ QSize BatteryMeter::sizeHint() const
 }
 
-void BatteryMeter::mouseReleaseEvent( QMouseEvent *)
+void BatteryMeter::mousePressEvent( QMouseEvent* e )
+{
+    if ( e->button() == RightButton )
+    {
+        style = 1-style;
+        Config c( "qpe" );
+        c.setGroup( "Battery" );
+        c.writeEntry( "Style", style );
+        repaint();
+    }
+    QWidget::mousePressEvent( e );
+}
+
+void BatteryMeter::mouseReleaseEvent( QMouseEvent* e)
 {
     if ( batteryView && batteryView->isVisible() ) {
 	delete (QWidget *) batteryView;
     } else {
-	if ( !batteryView )
-	    batteryView = new BatteryStatus( ps );
+        if ( !batteryView ) batteryView = new BatteryStatus( ps );
 	batteryView->showMaximized();
 	batteryView->raise();
@@ -99,24 +117,27 @@ void BatteryMeter::chargeTimeout()
 void BatteryMeter::paintEvent( QPaintEvent* )
 {
+    if ( style == 1 )
+    {
     QPainter p(this);
+        QFont f( "Fixed", AppLnk::smallIconSize()/2 );
+        QFontMetrics fm( f );
+        p.setFont( f );
+        p.drawText( 0, AppLnk::smallIconSize()/2, QString::number( percent ) );
+        p.drawText( AppLnk::smallIconSize()/4, AppLnk::smallIconSize(), "%" );
+        return;
+    }
 
-    QColor c;
-    QColor darkc;
-    QColor lightc;
-    if ( ps->acStatus() == PowerStatus::Offline ) {
-	c = blue.light(120);
-	darkc = c.dark(120);
-	lightc = c.light(140);
-    } else if ( ps->acStatus() == PowerStatus::Online ) {
-	c = green.dark(130);
-	darkc = c.dark(120);
-	lightc = c.light(180);
-    } else {
-	c = red;
-	darkc = c.dark(120);
-	lightc = c.light(160);
+    QPainter p(this);
+    QColor color;
+    QColor g = gray.light( 160 );
+    switch ( ps->acStatus() )
+    {
+        case PowerStatus::Offline: color = blue.light( 150 ); break;
+        case PowerStatus::Online: color = green.dark( 130 ).light( 180 ); break;
+        default: color = red.light( 160 );
     }
 
     int w = height() / 2 ;
+    if ( !(w%2) ) w--; // should have an odd value to get a real middle line
     int h = height() - 4;
     int pix = (percent * h) / 100;
@@ -126,28 +147,20 @@ void BatteryMeter::paintEvent( QPaintEvent* )
 
     p.setPen(QColor(80,80,80));
-    p.drawLine(x1+w/4,0,x1+w/4+w/2,0);
-    p.drawRect(x1,1,w,height()-1);
-    p.setBrush(c);
+    p.drawLine(x1+w/4,0,x1+w/4+w/2+1,0);  // header
+    p.drawRect(x1,1,w,height()-1);      // corpus
+    p.setBrush(color);
 
     int extra = ((percent * h) % 100)/(100/4);
 
-#define Y(i) ((i<=extra)?y-1:y)
-#define DRAWUPPER(i) if ( Y(i) >= 2 ) p.drawLine(i+x1,2,i+x1,Y(i));
-    p.setPen(  gray );
-    DRAWUPPER(1);
-    DRAWUPPER(3);
-    p.setPen( gray.light(130) );
-    DRAWUPPER(2);
-    p.setPen( gray.dark(120) );
-    DRAWUPPER(4);
-
-#define DRAW(i) { if ( Y(i) < y2 ) p.drawLine(i+x1,Y(i)+1, i+x1,y2); }
-    p.setPen( c );
-    DRAW(1);
-    DRAW(3);
-    p.setPen( lightc );
-    DRAW(2);
-    p.setPen(darkc);
-    DRAW(4);
+    int middle = w/2;
+    for ( int i = 0; i < middle; i++ )
+    {
+        p.setPen( gray.dark( 100+i*20 ) );
+        p.drawLine( x1+middle-i, 2, x1+middle-i, y-1 );
+        p.drawLine( x1+middle+i, 2, x1+middle+i, y-1 );
+        p.setPen( color.dark( 100+i*20 ) );
+        p.drawLine( x1+middle-i, y, x1+middle-i, y2 );
+        p.drawLine( x1+middle+i, y, x1+middle+i, y2 );
+    }
 }
 
diff --git a/core/applets/batteryapplet/battery.h b/core/applets/batteryapplet/battery.h
index d4807b0..c2d1216 100644
--- a/core/applets/batteryapplet/battery.h
+++ b/core/applets/batteryapplet/battery.h
@@ -40,4 +40,5 @@ protected:
     void timerEvent( QTimerEvent * );
     void paintEvent( QPaintEvent* );
+    void mousePressEvent( QMouseEvent* );
     void mouseReleaseEvent( QMouseEvent * );
 
@@ -51,4 +52,5 @@ protected:
     int percent;
     bool charging;
+    int style;
 };