- Improve look 'n feel.

- fix a bug in '=' operation.

5 files changed, 109 insertions, 33 deletions

diff --git a/noncore/tools/euroconv/calcdisplay.cpp b/noncore/tools/euroconv/calcdisplay.cpp
index 06e5158..1659858 100644
--- a/noncore/tools/euroconv/calcdisplay.cpp
+++ b/noncore/tools/euroconv/calcdisplay.cpp
@@ -17,2 +17,3 @@
 #include <qvbox.h>
+#include <qpixmap.h>
 
@@ -26,2 +27,6 @@ LCDDisplay::LCDDisplay( QWidget *parent, const char *name )
 
+
+this->setMargin(5);
+this->setSpacing(5);
+
 // Create display
@@ -29,4 +34,11 @@ QVBox *vbxlayout    = new QVBox (this);
 
-grpbxTop    = new QVGroupBox(vbxlayout, "grpbxTop");
+/***************    Top LCD   ***********************/
+grpbxTop    = new QHGroupBox(vbxlayout, "grpbxTop");
+grpbxStyle  = grpbxTop->frameStyle();
+grpbxTop->setMaximumHeight(48);
+
 cbbxTop     = new QComboBox(grpbxTop, "cbbxTop");
+cbbxTop->setMaximumWidth(50);
+cbbxTop->insertStrList(aCurrency);
+
 lcdTop      = new QLCDNumber(10, grpbxTop, "lcdTop");
@@ -35,6 +47,13 @@ lcdTop->setSmallDecimalPoint(true);
 lcdTop->setSegmentStyle(QLCDNumber::Flat);
-cbbxTop->insertStrList(aCurrency);
 
-grpbxBottom = new QVGroupBox(vbxlayout, "grpbxBottom");
+/**************   Bottom LCD  ************************/
+grpbxBottom = new QHGroupBox(vbxlayout, "grpbxBottom");
+grpbxBottom->setMaximumHeight(46);
+grpbxBottom->setFrameStyle(0);
+grpbxBottom->setFrameShadow(QFrame::MShadow);
+
 cbbxBottom  = new QComboBox(grpbxBottom, "cbbxBottom");
+cbbxBottom->setMaximumWidth(50);
+cbbxBottom->insertStrList(aCurrency);
+
 lcdBottom   = new QLCDNumber(10, grpbxBottom, "lcdBottom");
@@ -43,3 +62,2 @@ lcdBottom->setSmallDecimalPoint(true);
 lcdBottom->setSegmentStyle(QLCDNumber::Flat);
-cbbxBottom->insertStrList(aCurrency);
 
@@ -49,5 +67,6 @@ connect(cbbxBottom, SIGNAL(activated(int)), this, SLOT(cbbxChange()));
 
-btnSwap     = new QPushButton("S",this, "swap");
-btnSwap->setMaximumSize(20,50);
-btnSwap->setMinimumSize(20,50);
+btnSwap     = new QPushButton(this, "swap");
+QPixmap imgSwap((const char**) swap_xpm);
+btnSwap->setPixmap(imgSwap);
+btnSwap->setFixedSize(20,40);
 // set signal
@@ -58,4 +77,2 @@ iCurrentLCD = 0;
 
-//setValue(123.456);
-
 }
@@ -115,4 +132,8 @@ double dCurrentValue;
 if(!iCurrentLCD){
+    // iCurrentLCD = 0, lcdTop has current focus and is going to loose
+    // it
     dCurrentValue = lcdTop->value();
     iCurrentLCD = 1;
+    grpbxTop->setFrameStyle(0);
+    grpbxBottom->setFrameStyle(grpbxStyle);
 }else{
@@ -120,2 +141,4 @@ if(!iCurrentLCD){
     iCurrentLCD = 0;
+    grpbxTop->setFrameStyle(grpbxStyle);
+    grpbxBottom->setFrameStyle(0);
 }
@@ -132,4 +155,2 @@ double dCurrentValue;
 
-printf("combo changes...\n");
-
 // get current value
@@ -163,2 +184,5 @@ switch (iIndex){
         break;
+
+    default:
+        return 0;
 }//switch (iIndex)
@@ -187,2 +211,6 @@ switch (iIndex){
 }//switch (iIndex)
+
+// we shouldn't come here
+return 0;
+
 }// fct x2Euro
diff --git a/noncore/tools/euroconv/calcdisplay.h b/noncore/tools/euroconv/calcdisplay.h
index 17ce8f4..d1902cf 100644
--- a/noncore/tools/euroconv/calcdisplay.h
+++ b/noncore/tools/euroconv/calcdisplay.h
@@ -20,3 +20,3 @@
 #include <qlcdnumber.h>
-#include <qvgroupbox.h>
+#include <qhgroupbox.h>
 #include <qpushbutton.h>
@@ -24,2 +24,32 @@
 
+
+/* XPM */
+static char *swap_xpm[] = {
+/* width height num_colors chars_per_pixel */
+"    13    18        2            1",
+/* colors */
+". c None",
+"# c #000000",
+/* pixels */
+"..#######....",
+"..#####......",
+"..######.....",
+"..#...###....",
+"........##...",
+".........##..",
+"..........##.",
+"...........##",
+"...........##",
+"...........##",
+"...........##",
+"..........##.",
+".........##..",
+"........##...",
+"..#...###....",
+"..######.....",
+"..#####......",
+"..#######....",
+};
+
+
 class LCDDisplay : public QHBox{
@@ -38,3 +68,5 @@ public slots:
 private:
-    QVGroupBox  *grpbxTop;
+    int         grpbxStyle;
+
+    QHGroupBox  *grpbxTop;
     QComboBox   *cbbxTop;
@@ -42,3 +74,3 @@ private:
     
-    QVGroupBox  *grpbxBottom;
+    QHGroupBox  *grpbxBottom;
     QComboBox   *cbbxBottom;
diff --git a/noncore/tools/euroconv/calckeypad.cpp b/noncore/tools/euroconv/calckeypad.cpp
index 4156570..f1d61c0 100644
--- a/noncore/tools/euroconv/calckeypad.cpp
+++ b/noncore/tools/euroconv/calckeypad.cpp
@@ -23,2 +23,4 @@ KeyPad::KeyPad(LCDDisplay *lcd, QWidget *parent, const char *name )
 
+int         i;
+QButton *btnButton;
 
@@ -72,2 +74,8 @@ grpbtnDigits->insert(btn9, 9);
 
+// set appearance of buttons
+for(i=0; i<10; i++){
+    btnButton = grpbtnDigits->find(i);
+    btnButton->setFixedSize(30,30);
+}
+
 // Operator key
@@ -79,2 +87,13 @@ grpbtnOperators->insert(btnDivide,4);
 grpbtnOperators->insert(btnEqual,5);
+// set appearance of buttons
+for(i=1; i<6; i++){
+    btnButton = grpbtnOperators->find(i);
+    btnButton->setFixedSize(30,30);
+}
+btnClear->setFixedSize(30,30);
+btnClear->setPalette(QPalette( QColor(255, 99, 71) ) );
+btnDot->setFixedSize(30,30);
+btnPercent->setFixedSize(30,30);
+btnF1->setFixedSize(30,30);
+btnAbout->setFixedSize(30,30);
 
@@ -137,18 +156,11 @@ if(iPreviousOperator){
 }
-if(i == 5){
-    // key '=' pressed
-    dPreviousValue      = 0;
-    iPreviousOperator   = 0;
-    // We don't want to add digits
-    iLenCurrent         = 100;
-}else{
-    dPreviousValue      = dCurrent;
-    iPreviousOperator   = i;
-    // reset LCD for next digit
-    dCurrent            = 0;
-    iLenCurrent         = 1;
-    bIsDec              = false;
-    dDecCurrent         = 0;
-}
 
+dPreviousValue      = dCurrent;
+iPreviousOperator   = i;
+
+// reset LCD for next digit
+dCurrent            = 0;
+iLenCurrent         = 1;
+bIsDec              = false;
+dDecCurrent         = 0;
 }
@@ -189,3 +201,7 @@ QMessageBox::about( this, "About Euroconv",
                          "Written by Eric Santonacci for Opie\n"
-                         "http://opie.handhelds.org\n" );
+                         "http://opie.handhelds.org\n\n"
+                         "Version 0.2\n\n"
+                         "Any comment or feedback to:\n"
+                         "Eric.Santonacci@talc.fr\n");
+
 }
diff --git a/noncore/tools/euroconv/calckeypad.h b/noncore/tools/euroconv/calckeypad.h
index 42e7eb3..aac3f41 100644
--- a/noncore/tools/euroconv/calckeypad.h
+++ b/noncore/tools/euroconv/calckeypad.h
@@ -73,3 +73,3 @@ private:
 
- 
+    void setAppearance(void);
 
diff --git a/noncore/tools/euroconv/currency.h b/noncore/tools/euroconv/currency.h
index 90f5b57..d9ee75c 100644
--- a/noncore/tools/euroconv/currency.h
+++ b/noncore/tools/euroconv/currency.h
@@ -17,4 +17,4 @@
 #define FF      6.55957         // French Francs
-#define DM      1.96            // Deutch Mark
+#define DM      1.9594          // Deutch Mark
 
-static const char* aCurrency[] = { "Euro", "French Frs", "DM", 0 };
+static const char* aCurrency[] = { "Euro", "FF", "DM", 0 };