From 5ce3a5c21389d57087a1d9a36cf45411438d5a96 Mon Sep 17 00:00:00 2001 From: arcan1s Date: Wed, 19 Nov 2014 11:14:55 +0300 Subject: [PATCH 1/5] fix configuration --- sources/awesome-widget/awesome-widget.cpp | 2 +- sources/awesome-widget/graphicalitem.cpp | 2 +- sources/ext-sysmon/extscript.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sources/awesome-widget/awesome-widget.cpp b/sources/awesome-widget/awesome-widget.cpp index 3f36fac..dfe34c6 100644 --- a/sources/awesome-widget/awesome-widget.cpp +++ b/sources/awesome-widget/awesome-widget.cpp @@ -418,7 +418,7 @@ void AwesomeWidget::getGraphicalItems() graphicalItems.clear(); // create directory at $HOME - QString localDir = KStandardDirs::locateLocal("data", "plasma_applet_awesome-widget/destkops"); + QString localDir = KStandardDirs::locateLocal("data", "plasma_applet_awesome-widget/desktops"); if (KStandardDirs::makeDir(localDir)) if (debug) qDebug() << PDEBUG << ":" << "Created directory" << localDir; diff --git a/sources/awesome-widget/graphicalitem.cpp b/sources/awesome-widget/graphicalitem.cpp index 46e23e1..a268ce9 100644 --- a/sources/awesome-widget/graphicalitem.cpp +++ b/sources/awesome-widget/graphicalitem.cpp @@ -362,7 +362,7 @@ void GraphicalItem::showConfiguration(const QStringList tags) ui->spinBox_width->setValue(_width); int ret = exec(); - if (ret == 0) { + if (ret == 1) { QStringList colorText; setName(ui->label_nameValue->text()); setComment(ui->lineEdit_comment->text()); diff --git a/sources/ext-sysmon/extscript.cpp b/sources/ext-sysmon/extscript.cpp index d9cb78d..939451b 100644 --- a/sources/ext-sysmon/extscript.cpp +++ b/sources/ext-sysmon/extscript.cpp @@ -295,7 +295,7 @@ void ExtScript::showConfiguration() ui->spinBox_interval->setValue(_interval); int ret = exec(); - if (ret == 0) { + if (ret == 1) { setName(ui->lineEdit_name->text()); setComment(ui->lineEdit_comment->text()); setExec(ui->lineEdit_command->text()); From c3420a8393a487091213540f0f80abb796d116ca Mon Sep 17 00:00:00 2001 From: arcan1s Date: Wed, 19 Nov 2014 12:08:54 +0300 Subject: [PATCH 2/5] some simplification of graphicalitem --- sources/awesome-widget/graphicalitem.cpp | 166 ++++++++++++----------- sources/awesome-widget/graphicalitem.h | 15 +- 2 files changed, 96 insertions(+), 85 deletions(-) diff --git a/sources/awesome-widget/graphicalitem.cpp b/sources/awesome-widget/graphicalitem.cpp index a268ce9..d61551e 100644 --- a/sources/awesome-widget/graphicalitem.cpp +++ b/sources/awesome-widget/graphicalitem.cpp @@ -59,6 +59,8 @@ QString GraphicalItem::getImage(const float value) if (debug) qDebug() << PDEBUG << ":" << "Value" << value; if (_bar == QString("none")) return QString(""); + QColor active = stringToColor(_activeColor); + QColor inactive = stringToColor(_inactiveColor); float percent = value / 100.0; int scale[2] = {1, 1}; QPen pen = QPen(); @@ -76,10 +78,10 @@ QString GraphicalItem::getImage(const float value) case Vertical: pen.setWidth(_width); // inactive - pen.setColor(_inactiveColor); + pen.setColor(inactive); scene->addLine(0.5 * _width, 0.0, 0.5 * _width, _height, pen); // active - pen.setColor(_activeColor); + pen.setColor(active); scene->addLine(0.5 * _width, (1.0 - percent) * _height + 0.5 * _width, 0.5 * _width, _height + 0.5 * _width, pen); // scale scale[1] = (int)_direction; @@ -88,12 +90,12 @@ QString GraphicalItem::getImage(const float value) QGraphicsEllipseItem *circle; pen.setWidth(1.0); // inactive - pen.setColor(_inactiveColor); + pen.setColor(inactive); circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(_inactiveColor, Qt::SolidPattern)); circle->setSpanAngle((1.0 - percent) * 360.0 * 16.0); circle->setStartAngle(180.0 * 16.0 - (1.0 - percent) * 360.0 * 16.0); // active - pen.setColor(_activeColor); + pen.setColor(active); circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(_activeColor, Qt::SolidPattern)); circle->setSpanAngle(percent * 360.0 * 16.0); circle->setStartAngle(180.0 * 16.0); @@ -103,10 +105,10 @@ QString GraphicalItem::getImage(const float value) default: pen.setWidth(_height); // inactive - pen.setColor(_inactiveColor); + pen.setColor(inactive); scene->addLine(0.0, 0.5 * _height, _width, 0.5 * _height, pen); // active - pen.setColor(_activeColor); + pen.setColor(active); scene->addLine(-0.5 * _height, 0.5 * _height, percent * _width - 0.5 * _height, 0.5 * _height, pen); // scale scale[0] = (int)_direction; @@ -158,7 +160,7 @@ QString GraphicalItem::getBar() } -QColor GraphicalItem::getActiveColor() +QString GraphicalItem::getActiveColor() { if (debug) qDebug() << PDEBUG; @@ -166,7 +168,7 @@ QColor GraphicalItem::getActiveColor() } -QColor GraphicalItem::getInactiveColor() +QString GraphicalItem::getInactiveColor() { if (debug) qDebug() << PDEBUG; @@ -182,6 +184,27 @@ GraphicalItem::Type GraphicalItem::getType() } +QString GraphicalItem::getStrType() +{ + if (debug) qDebug() << PDEBUG; + + QString value; + switch(_type) { + case Vertical: + value = QString("Vertical"); + break; + case Circle: + value = QString("Circle"); + break; + default: + value = QString("Horizontal"); + break; + }\ + + return value; +} + + GraphicalItem::Direction GraphicalItem::getDirection() { if (debug) qDebug() << PDEBUG; @@ -190,6 +213,24 @@ GraphicalItem::Direction GraphicalItem::getDirection() } +QString GraphicalItem::getStrDirection() +{ + if (debug) qDebug() << PDEBUG; + + QString value; + switch (_direction) { + case RightToLeft: + value = QString("RightToLeft"); + break; + default: + value = QString("LeftToRight"); + break; + } + + return value; +} + + int GraphicalItem::getHeight() { if (debug) qDebug() << PDEBUG; @@ -240,7 +281,7 @@ void GraphicalItem::setBar(const QString bar) } -void GraphicalItem::setActiveColor(const QColor color) +void GraphicalItem::setActiveColor(const QString color) { if (debug) qDebug() << PDEBUG; if (debug) qDebug() << PDEBUG << ":" << "Color" << color; @@ -249,7 +290,7 @@ void GraphicalItem::setActiveColor(const QColor color) } -void GraphicalItem::setInactiveColor(const QColor color) +void GraphicalItem::setInactiveColor(const QString color) { if (debug) qDebug() << PDEBUG; if (debug) qDebug() << PDEBUG << ":" << "Color" << color; @@ -312,30 +353,15 @@ void GraphicalItem::readConfiguration() if (!QDir(dirs[i]).entryList(QDir::Files).contains(fileName)) continue; QSettings settings(dirs[i] + QDir::separator() + fileName, QSettings::IniFormat); settings.beginGroup(QString("Desktop Entry")); - QStringList childKeys = settings.childKeys(); - for (int i=0; icomboBox_value->addItems(tags); ui->comboBox_value->addItem(_bar); ui->comboBox_value->setCurrentIndex(ui->comboBox_value->count() - 1); - ui->pushButton_activeColor->setText(QString("%1,%2,%3,%4") - .arg(_activeColor.red()).arg(_activeColor.green()) - .arg(_activeColor.blue()).arg(_activeColor.alpha())); - ui->pushButton_inactiveColor->setText(QString("%1,%2,%3,%4") - .arg(_inactiveColor.red()).arg(_inactiveColor.green()) - .arg(_inactiveColor.blue()).arg(_inactiveColor.alpha())); + ui->pushButton_activeColor->setText(_activeColor); + ui->pushButton_inactiveColor->setText(_inactiveColor); ui->comboBox_type->setCurrentIndex((int)_type); ui->comboBox_direction->setCurrentIndex((int)_direction); ui->spinBox_height->setValue(_height); @@ -363,16 +385,11 @@ void GraphicalItem::showConfiguration(const QStringList tags) int ret = exec(); if (ret == 1) { - QStringList colorText; setName(ui->label_nameValue->text()); setComment(ui->lineEdit_comment->text()); setBar(ui->comboBox_value->currentText()); - colorText = ui->pushButton_activeColor->text().split(QChar(',')); - setActiveColor(QColor(colorText[0].toInt(), colorText[1].toInt(), - colorText[2].toInt(), colorText[3].toInt())); - colorText = ui->pushButton_inactiveColor->text().split(QChar(',')); - setInactiveColor(QColor(colorText[0].toInt(), colorText[1].toInt(), - colorText[2].toInt(), colorText[3].toInt())); + setActiveColor(ui->pushButton_activeColor->text()); + setInactiveColor(ui->pushButton_inactiveColor->text()); setType(ui->comboBox_type->currentText()); setDirection(ui->comboBox_direction->currentText()); setHeight(ui->spinBox_height->value()); @@ -400,38 +417,14 @@ void GraphicalItem::writeConfiguration() if (debug) qDebug() << PDEBUG << ":" << "Configuration file" << settings.fileName(); settings.beginGroup(QString("Desktop Entry")); - QString strValue; settings.setValue(QString("Encoding"), QString("UTF-8")); settings.setValue(QString("Name"), _name); settings.setValue(QString("Comment"), _comment); settings.setValue(QString("X-AW-Value"), _bar); - settings.setValue(QString("X-AW-ActiveColor"), QString("%1,%2,%3,%4") - .arg(_activeColor.red()).arg(_activeColor.green()) - .arg(_activeColor.blue()).arg(_activeColor.alpha())); - settings.setValue(QString("X-AW-InactiveColor"), QString("%1,%2,%3,%4") - .arg(_inactiveColor.red()).arg(_inactiveColor.green()) - .arg(_inactiveColor.blue()).arg(_inactiveColor.alpha())); - switch(_type) { - case Vertical: - strValue = QString("Vertical"); - break; - case Circle: - strValue = QString("Circle"); - break; - default: - strValue = QString("Horizontal"); - break; - } - settings.setValue(QString("X-AW-Type"), strValue); - switch (_direction) { - case RightToLeft: - strValue = QString("RightToLeft"); - break; - default: - strValue = QString("LeftToRight"); - break; - } - settings.setValue(QString("X-AW-Direction"), strValue); + settings.setValue(QString("X-AW-ActiveColor"), _activeColor); + settings.setValue(QString("X-AW-InactiveColor"), _inactiveColor); + settings.setValue(QString("X-AW-Type"), getStrType()); + settings.setValue(QString("X-AW-Direction"), getStrDirection()); settings.setValue(QString("X-AW-Height"), _height); settings.setValue(QString("X-AW-Width"), _width); @@ -444,16 +437,31 @@ void GraphicalItem::changeColor() { if (debug) qDebug() << PDEBUG; - QStringList colorText = ((QPushButton *)sender())->text().split(QChar(',')); - QColor color = QColor(colorText[0].toInt(), colorText[1].toInt(), - colorText[2].toInt(), colorText[3].toInt()); + QColor color = stringToColor(((QPushButton *)sender())->text()); QColor newColor = QColorDialog::getColor(color, 0, i18n("Select color"), QColorDialog::ShowAlphaChannel); - colorText.clear(); + QStringList colorText; colorText.append(QString("%1").arg(newColor.red())); colorText.append(QString("%1").arg(newColor.green())); colorText.append(QString("%1").arg(newColor.blue())); colorText.append(QString("%1").arg(newColor.alpha())); ((QPushButton *)sender())->setText(colorText.join(QChar(','))); } + + +QColor GraphicalItem::stringToColor(const QString color) +{ + if (debug) qDebug() << PDEBUG; + + QColor qcolor; + QStringList listColor = color.split(QChar(',')); + while (listColor.count() < 4) + listColor.append(QString("0")); + qcolor.setRed(listColor[0].toInt()); + qcolor.setGreen(listColor[1].toInt()); + qcolor.setBlue(listColor[2].toInt()); + qcolor.setAlpha(listColor[3].toInt()); + + return qcolor; +} diff --git a/sources/awesome-widget/graphicalitem.h b/sources/awesome-widget/graphicalitem.h index b9e538f..777d488 100644 --- a/sources/awesome-widget/graphicalitem.h +++ b/sources/awesome-widget/graphicalitem.h @@ -49,18 +49,20 @@ public: QString getName(); QString getComment(); QString getBar(); - QColor getActiveColor(); - QColor getInactiveColor(); + QString getActiveColor(); + QString getInactiveColor(); Type getType(); + QString getStrType(); Direction getDirection(); + QString getStrDirection(); int getHeight(); int getWidth(); // set methods void setName(const QString name = QString("none")); void setComment(const QString comment = QString("empty")); void setBar(const QString bar = QString("cpu")); - void setActiveColor(const QColor color = QColor(255, 255, 255, 0)); - void setInactiveColor(const QColor color = QColor(255, 255, 255, 255)); + void setActiveColor(const QString color = QString("0,0,0,130")); + void setInactiveColor(const QString color = QString("255,255,255,130")); void setType(const QString type = QString("Horizontal")); void setDirection(const QString direction = QString("LeftToRight")); void setHeight(const int height = 100); @@ -76,6 +78,7 @@ private slots: void changeColor(); private: + QColor stringToColor(const QString color); QString fileName; QStringList dirs; bool debug; @@ -84,8 +87,8 @@ private: QString _name = QString("none"); QString _comment = QString("empty"); QString _bar = QString("cpu"); - QColor _activeColor = QColor(255, 255, 255, 0); - QColor _inactiveColor = QColor(255, 255, 255, 255); + QString _activeColor = QString("0,0,0,130"); + QString _inactiveColor = QString("255,255,255,130"); Type _type = Horizontal; Direction _direction = LeftToRight; int _height = 100; From 5f37a9d8c712c242fadc520dfb12549581acecad Mon Sep 17 00:00:00 2001 From: arcan1s Date: Wed, 19 Nov 2014 12:28:43 +0300 Subject: [PATCH 3/5] fix direction --- sources/awesome-widget/graphicalitem.cpp | 14 +++++++------- sources/awesome-widget/graphicalitem.h | 4 ++-- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/sources/awesome-widget/graphicalitem.cpp b/sources/awesome-widget/graphicalitem.cpp index d61551e..0733cce 100644 --- a/sources/awesome-widget/graphicalitem.cpp +++ b/sources/awesome-widget/graphicalitem.cpp @@ -79,39 +79,39 @@ QString GraphicalItem::getImage(const float value) pen.setWidth(_width); // inactive pen.setColor(inactive); - scene->addLine(0.5 * _width, 0.0, 0.5 * _width, _height, pen); + scene->addLine(0.5 * _width, -0.5 * _width, 0.5 * _width, (1.0 - percent) * _height - 0.5 * _width, pen); // active pen.setColor(active); scene->addLine(0.5 * _width, (1.0 - percent) * _height + 0.5 * _width, 0.5 * _width, _height + 0.5 * _width, pen); // scale - scale[1] = (int)_direction; + scale[1] = -2 * (int)_direction + 1; break; case Circle: QGraphicsEllipseItem *circle; pen.setWidth(1.0); // inactive pen.setColor(inactive); - circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(_inactiveColor, Qt::SolidPattern)); + circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(inactive, Qt::SolidPattern)); circle->setSpanAngle((1.0 - percent) * 360.0 * 16.0); circle->setStartAngle(180.0 * 16.0 - (1.0 - percent) * 360.0 * 16.0); // active pen.setColor(active); - circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(_activeColor, Qt::SolidPattern)); + circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(active, Qt::SolidPattern)); circle->setSpanAngle(percent * 360.0 * 16.0); circle->setStartAngle(180.0 * 16.0); // scale - scale[0] = (int)_direction; + scale[0] = -2 *(int)_direction + 1; break; default: pen.setWidth(_height); // inactive pen.setColor(inactive); - scene->addLine(0.0, 0.5 * _height, _width, 0.5 * _height, pen); + scene->addLine(percent * _width + 0.5 * _height, 0.5 * _height, _width + 0.5 * _height, 0.5 * _height, pen); // active pen.setColor(active); scene->addLine(-0.5 * _height, 0.5 * _height, percent * _width - 0.5 * _height, 0.5 * _height, pen); // scale - scale[0] = (int)_direction; + scale[0] = -2 * (int)_direction + 1; break; } diff --git a/sources/awesome-widget/graphicalitem.h b/sources/awesome-widget/graphicalitem.h index 777d488..1795393 100644 --- a/sources/awesome-widget/graphicalitem.h +++ b/sources/awesome-widget/graphicalitem.h @@ -31,8 +31,8 @@ class GraphicalItem : public QDialog public: enum Direction { - RightToLeft = 0, - LeftToRight = 1 + LeftToRight = 0, + RightToLeft }; enum Type { Horizontal = 0, From 2c3c5754939c40cec4243117b33587b413289195 Mon Sep 17 00:00:00 2001 From: arcan1s Date: Wed, 19 Nov 2014 15:04:10 +0300 Subject: [PATCH 4/5] * add .desktop suffix to files if they have not * remove frameshape --- sources/awesome-widget/configuration.cpp | 2 ++ sources/awesome-widget/graphicalitem.cpp | 1 + 2 files changed, 3 insertions(+) diff --git a/sources/awesome-widget/configuration.cpp b/sources/awesome-widget/configuration.cpp index eb5be8f..582e032 100644 --- a/sources/awesome-widget/configuration.cpp +++ b/sources/awesome-widget/configuration.cpp @@ -722,6 +722,7 @@ void AwesomeWidget::addBar() i18n("File name"), QLineEdit::Normal, QString(""), &ok); if ((!ok) || (name.isEmpty())) return; + if (!name.endsWith(QString(".desktop"))) name += QString(".desktop"); QStringList bars; bars.append(keys.filter((QRegExp(QString("cpu(?!cl).*"))))); bars.append(keys.filter((QRegExp(QString("gpu"))))); @@ -746,6 +747,7 @@ void AwesomeWidget::addCustomScript() i18n("File name"), QLineEdit::Normal, QString(""), &ok); if ((!ok) || (name.isEmpty())) return; + if (!name.endsWith(QString(".desktop"))) name += QString(".desktop"); ExtScript *script = new ExtScript(0, name, dirs, debug); script->showConfiguration(); diff --git a/sources/awesome-widget/graphicalitem.cpp b/sources/awesome-widget/graphicalitem.cpp index 0733cce..c67424a 100644 --- a/sources/awesome-widget/graphicalitem.cpp +++ b/sources/awesome-widget/graphicalitem.cpp @@ -69,6 +69,7 @@ QString GraphicalItem::getImage(const float value) QGraphicsView *view = new QGraphicsView(scene); view->setStyleSheet(QString("background: transparent")); view->setContentsMargins(0, 0, 0, 0); + view->setFrameShape(QFrame::NoFrame); view->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff); view->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff); view->resize(_width + 5.0, _height + 5.0); From ca60a8524185c8e5defa8e6154d15974cf612579 Mon Sep 17 00:00:00 2001 From: arcan1s Date: Wed, 19 Nov 2014 16:19:23 +0300 Subject: [PATCH 5/5] update graphicalitem --- sources/awesome-widget/configuration.cpp | 22 ++++++++-------- sources/awesome-widget/graphicalitem.cpp | 33 ++++++++++++------------ sources/awesome-widget/graphicalitem.ui | 10 +++---- 3 files changed, 33 insertions(+), 32 deletions(-) diff --git a/sources/awesome-widget/configuration.cpp b/sources/awesome-widget/configuration.cpp index 582e032..a9e97c5 100644 --- a/sources/awesome-widget/configuration.cpp +++ b/sources/awesome-widget/configuration.cpp @@ -725,11 +725,11 @@ void AwesomeWidget::addBar() if (!name.endsWith(QString(".desktop"))) name += QString(".desktop"); QStringList bars; bars.append(keys.filter((QRegExp(QString("cpu(?!cl).*"))))); - bars.append(keys.filter((QRegExp(QString("gpu"))))); - bars.append(keys.filter((QRegExp(QString("mem"))))); - bars.append(keys.filter((QRegExp(QString("swap"))))); - bars.append(keys.filter((QRegExp(QString("hdd[0-9].*"))))); - bars.append(keys.filter((QRegExp(QString("bat.*"))))); + bars.append(keys.filter((QRegExp(QString("^gpu$"))))); + bars.append(keys.filter((QRegExp(QString("^mem$"))))); + bars.append(keys.filter((QRegExp(QString("^swap$"))))); + bars.append(keys.filter((QRegExp(QString("^hdd[0-9].*"))))); + bars.append(keys.filter((QRegExp(QString("^bat.*"))))); GraphicalItem *item = new GraphicalItem(0, name, dirs, debug); item->setName(QString("bar%1").arg(number)); @@ -832,12 +832,12 @@ void AwesomeWidget::editBar(QListWidgetItem *item) if (debug) qDebug() << PDEBUG; QStringList bars; - bars.append(keys.filter((QRegExp(QString("cpu(?!cl).*"))))); - bars.append(keys.filter((QRegExp(QString("gpu$"))))); - bars.append(keys.filter((QRegExp(QString("mem$"))))); - bars.append(keys.filter((QRegExp(QString("swap$"))))); - bars.append(keys.filter((QRegExp(QString("hdd[0-9].*"))))); - bars.append(keys.filter((QRegExp(QString("bat.*"))))); + bars.append(keys.filter((QRegExp(QString("^cpu(?!cl).*"))))); + bars.append(keys.filter((QRegExp(QString("^gpu$"))))); + bars.append(keys.filter((QRegExp(QString("^mem$"))))); + bars.append(keys.filter((QRegExp(QString("^swap$"))))); + bars.append(keys.filter((QRegExp(QString("^hdd[0-9].*"))))); + bars.append(keys.filter((QRegExp(QString("^bat.*"))))); for (int i=0; igetFileName() != item->text()) continue; diff --git a/sources/awesome-widget/graphicalitem.cpp b/sources/awesome-widget/graphicalitem.cpp index c67424a..3e7a418 100644 --- a/sources/awesome-widget/graphicalitem.cpp +++ b/sources/awesome-widget/graphicalitem.cpp @@ -93,13 +93,13 @@ QString GraphicalItem::getImage(const float value) // inactive pen.setColor(inactive); circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(inactive, Qt::SolidPattern)); - circle->setSpanAngle((1.0 - percent) * 360.0 * 16.0); - circle->setStartAngle(180.0 * 16.0 - (1.0 - percent) * 360.0 * 16.0); + circle->setSpanAngle(- (1.0 - percent) * 360.0 * 16.0); + circle->setStartAngle(90.0 * 16.0 - percent * 360.0 * 16.0); // active pen.setColor(active); circle = scene->addEllipse(0.0, 0.0, _width, _height, pen, QBrush(active, Qt::SolidPattern)); - circle->setSpanAngle(percent * 360.0 * 16.0); - circle->setStartAngle(180.0 * 16.0); + circle->setSpanAngle(- percent * 360.0 * 16.0); + circle->setStartAngle(90.0 * 16.0); // scale scale[0] = -2 *(int)_direction + 1; break; @@ -385,18 +385,17 @@ void GraphicalItem::showConfiguration(const QStringList tags) ui->spinBox_width->setValue(_width); int ret = exec(); - if (ret == 1) { - setName(ui->label_nameValue->text()); - setComment(ui->lineEdit_comment->text()); - setBar(ui->comboBox_value->currentText()); - setActiveColor(ui->pushButton_activeColor->text()); - setInactiveColor(ui->pushButton_inactiveColor->text()); - setType(ui->comboBox_type->currentText()); - setDirection(ui->comboBox_direction->currentText()); - setHeight(ui->spinBox_height->value()); - setWidth(ui->spinBox_width->value()); - writeConfiguration(); - } + if (ret != 1) return; + setName(ui->label_nameValue->text()); + setComment(ui->lineEdit_comment->text()); + setBar(ui->comboBox_value->currentText()); + setActiveColor(ui->pushButton_activeColor->text().remove(QChar('&'))); + setInactiveColor(ui->pushButton_inactiveColor->text().remove(QChar('&'))); + setType(ui->comboBox_type->currentText()); + setDirection(ui->comboBox_direction->currentText()); + setHeight(ui->spinBox_height->value()); + setWidth(ui->spinBox_width->value()); + writeConfiguration(); } @@ -441,6 +440,7 @@ void GraphicalItem::changeColor() QColor color = stringToColor(((QPushButton *)sender())->text()); QColor newColor = QColorDialog::getColor(color, 0, i18n("Select color"), QColorDialog::ShowAlphaChannel); + if (!newColor.isValid()) return; QStringList colorText; colorText.append(QString("%1").arg(newColor.red())); @@ -454,6 +454,7 @@ void GraphicalItem::changeColor() QColor GraphicalItem::stringToColor(const QString color) { if (debug) qDebug() << PDEBUG; + if (debug) qDebug() << PDEBUG << ":" << "Color" << color; QColor qcolor; QStringList listColor = color.split(QChar(',')); diff --git a/sources/awesome-widget/graphicalitem.ui b/sources/awesome-widget/graphicalitem.ui index 03693b8..4f66036 100644 --- a/sources/awesome-widget/graphicalitem.ui +++ b/sources/awesome-widget/graphicalitem.ui @@ -6,8 +6,8 @@ 0 0 - 420 - 329 + 416 + 325 @@ -73,7 +73,7 @@ - true + false @@ -94,7 +94,7 @@ - + @@ -115,7 +115,7 @@ - +