summaryrefslogtreecommitdiff
authorleseb <leseb>2002-04-30 08:54:28 (UTC)
committer leseb <leseb>2002-04-30 08:54:28 (UTC)
commit713aa4385e859c9300ee2a5b0086fbaabf3b3a33 (patch) (side-by-side diff)
treeee835511cd9e024ccb1ceb4fa19e7be5db71ff9e
parentace7b43b1c978950ca38e8bd962f4b52d34ac307 (diff)
downloadopie-713aa4385e859c9300ee2a5b0086fbaabf3b3a33.zip
opie-713aa4385e859c9300ee2a5b0086fbaabf3b3a33.tar.gz
opie-713aa4385e859c9300ee2a5b0086fbaabf3b3a33.tar.bz2
Clean code + fix 2 new introduced bugs
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--noncore/graphics/drawpad/drawpad.cpp58
1 files changed, 14 insertions, 44 deletions
diff --git a/noncore/graphics/drawpad/drawpad.cpp b/noncore/graphics/drawpad/drawpad.cpp
index 77fee98..74787e5 100644
--- a/noncore/graphics/drawpad/drawpad.cpp
+++ b/noncore/graphics/drawpad/drawpad.cpp
@@ -13,8 +13,6 @@
#include "drawpad.h"
-//#include "colordialog.h"
-//#include "colorpanel.h"
#include "drawpadcanvas.h"
#include "ellipsetool.h"
#include "erasetool.h"
@@ -28,13 +26,14 @@
#include "rectangletool.h"
#include "texttool.h"
+#include <opie/colordialog.h>
+#include <opie/colorpopupmenu.h>
+
#include <qpe/applnk.h>
#include <qpe/global.h>
#include <qpe/qpemenubar.h>
#include <qpe/qpetoolbar.h>
#include <qpe/resource.h>
-#include <opie/colordialog.h>
-#include <opie/colorpopupmenu.h>
#include <qaction.h>
#include <qfile.h>
@@ -218,55 +217,26 @@ DrawPad::DrawPad(QWidget* parent, const char* name)
m_pPenColorToolButton = new QToolButton(drawParametersToolBar);
m_pPenColorToolButton->setPixmap(Resource::loadPixmap("drawpad/pencolor.png"));
- ColorPopupMenu* colorPopupMenu = new ColorPopupMenu( Qt::black, m_pPenColorToolButton );
- m_pPenColorToolButton->setPopup( colorPopupMenu );
- m_pPenColorToolButton->setPopupDelay( 0 );
- QToolTip::add(m_pPenColorToolButton, tr("Pen Color"));
- connect( colorPopupMenu, SIGNAL( colorSelected( const QColor& ) ), this, SLOT( changePenColor( const QColor& ) ) );
-
-// QPopupMenu* penColorPopupMenu = new QPopupMenu(m_pPenColorToolButton);
-
-// ColorPanel* penColorPanel = new ColorPanel(penColorPopupMenu);
-// connect(penColorPanel, SIGNAL(colorSelected(const QColor&)), this, SLOT(changePenColor(const QColor&)));
-
-// penColorPopupMenu->insertItem(penColorPanel);
-// penColorPopupMenu->insertSeparator();
+ ColorPopupMenu* penColorPopupMenu = new ColorPopupMenu(Qt::black, m_pPenColorToolButton);
+ connect(penColorPopupMenu, SIGNAL(colorSelected(const QColor&)), this, SLOT(changePenColor(const QColor&)));
-// QAction* choosePenColorAction = new QAction(tr("More"), tr("More..."), 0, this);
-// connect(choosePenColorAction, SIGNAL(activated()), this, SLOT(choosePenColor()));
-// choosePenColorAction->addTo(penColorPopupMenu);
-
-// m_pPenColorToolButton->setPopup(colorPopupMenu);
-// m_pPenColorToolButton->setPopupDelay(0);
+ QToolTip::add(m_pPenColorToolButton, tr("Pen Color"));
+ m_pPenColorToolButton->setPopup(penColorPopupMenu);
+ m_pPenColorToolButton->setPopupDelay(0);
-// penColorPanel->buttonSelected(Qt::black);
+ changePenColor(Qt::black);
m_pBrushColorToolButton = new QToolButton(drawParametersToolBar);
m_pBrushColorToolButton->setPixmap(Resource::loadPixmap("drawpad/brushcolor.png"));
+ ColorPopupMenu* brushColorPopupMenu = new ColorPopupMenu(Qt::white, m_pBrushColorToolButton);
+ connect(brushColorPopupMenu, SIGNAL(colorSelected(const QColor&)), this, SLOT(changeBrushColor(const QColor&)));
- colorPopupMenu = new ColorPopupMenu( Qt::white, m_pBrushColorToolButton );
- m_pBrushColorToolButton->setPopup( colorPopupMenu );
- m_pBrushColorToolButton->setPopupDelay( 0 );
QToolTip::add(m_pBrushColorToolButton, tr("Fill Color"));
- connect( colorPopupMenu, SIGNAL( colorSelected( const QColor& ) ), this, SLOT( changeeBrushColor( const QColor& ) ) );
-
-// QPopupMenu* brushColorPopupMenu = new QPopupMenu(m_pBrushColorToolButton);
-
-// ColorPanel* brushColorPanel = new ColorPanel(brushColorPopupMenu);
-// connect(brushColorPanel, SIGNAL(colorSelected(const QColor&)), this, SLOT(changeBrushColor(const QColor&)));
-
-// brushColorPopupMenu->insertItem(brushColorPanel);
-// brushColorPopupMenu->insertSeparator();
-
-// QAction* chooseBrushColorAction = new QAction(tr("More"), tr("More..."), 0, this);
-// connect(chooseBrushColorAction, SIGNAL(activated()), this, SLOT(chooseBrushColor()));
-// chooseBrushColorAction->addTo(brushColorPopupMenu);
-
-// m_pBrushColorToolButton->setPopup(brushColorPopupMenu);
-// m_pBrushColorToolButton->setPopupDelay(0);
+ m_pBrushColorToolButton->setPopup(brushColorPopupMenu);
+ m_pBrushColorToolButton->setPopupDelay(0);
-// brushColorPanel->buttonSelected(Qt::white);
+ changeBrushColor(Qt::white);
// init pages