Remove highlight-color related settings from QtUiStyle[Settings]
authorManuel Nickschas <sputnick@quassel-irc.org>
Sun, 21 Jun 2009 19:58:18 +0000 (21:58 +0200)
committerManuel Nickschas <sputnick@quassel-irc.org>
Thu, 6 Aug 2009 18:25:05 +0000 (20:25 +0200)
Since we now use message labels for styling highlighted messages, it's no longer
necessary to have explicit options for this in QtUiStyle.

src/qtui/qtuisettings.cpp
src/qtui/qtuisettings.h
src/qtui/qtuistyle.cpp
src/qtui/qtuistyle.h
src/qtui/settingspages/colorsettingspage.cpp

index 8141154..3cdd26a 100644 (file)
@@ -40,11 +40,3 @@ QtUiStyleSettings::QtUiStyleSettings()
   : UiStyleSettings("QtUiStyle")
 {
 }
-
-void QtUiStyleSettings::setHighlightColor(const QColor &col) {
-  setLocalValue("Colors/HighlightColor", col);
-}
-
-QColor QtUiStyleSettings::highlightColor() {
-  return localValue("Colors/HighlightColor", QColor()).value<QColor>();
-}
index 4d199fd..8559c6e 100644 (file)
@@ -35,9 +35,6 @@ class QtUiStyleSettings : public UiStyleSettings {
 public:
   QtUiStyleSettings(const QString &subGroup);
   QtUiStyleSettings();
-
-  void setHighlightColor(const QColor &);
-  QColor highlightColor();
 };
 
 #endif
index 5ea72c1..5aa6534 100644 (file)
@@ -26,9 +26,3 @@ QtUiStyle::QtUiStyle() : UiStyle() {
 }
 
 QtUiStyle::~QtUiStyle() {}
-
-void QtUiStyle::setHighlightColor(const QColor &col) {
-  _highlightColor = col;
-  QtUiStyleSettings s;
-  s.setHighlightColor(col);
-}
index 270e248..e341736 100644 (file)
@@ -31,11 +31,7 @@ public:
 
   virtual inline qreal firstColumnSeparator() const { return 6; }
   virtual inline qreal secondColumnSeparator() const { return 6; }
-  virtual inline QColor highlightColor() const { return _highlightColor; }
-  virtual void setHighlightColor(const QColor &);
 
-private:
-  QColor _highlightColor;
 };
 
 #endif
index f77d578..58f699a 100644 (file)
@@ -362,9 +362,9 @@ void ColorSettingsPage::load() {
   ui.modeFlagsBG->setColor(QtUi::style()->format(UiStyle::ModeFlags).background().color());
   ui.urlFG->setColor(QtUi::style()->format(UiStyle::Url).foreground().color());
   ui.urlBG->setColor(QtUi::style()->format(UiStyle::Url).background().color());
-*/
+
   ui.highlightColor->setColor(QtUi::style()->highlightColor());
-/*
+
   ui.color0->setColor(QtUi::style()->format(UiStyle::FgCol00).foreground().color());
   ui.color1->setColor(QtUi::style()->format(UiStyle::FgCol01).foreground().color());
   ui.color2->setColor(QtUi::style()->format(UiStyle::FgCol02).foreground().color());
@@ -444,7 +444,7 @@ void ColorSettingsPage::save() {
   saveColor(UiStyle::NickMsg, ui.renameMessageFG->color(), ui.renameMessageBG->color(), ui.renameMessageUseBG->isChecked());
   s.setValue("renameMessageUseBG", ui.renameMessageUseBG->isChecked());
 
-  QtUi::style()->setHighlightColor(ui.highlightColor->color());
+  //QtUi::style()->setHighlightColor(ui.highlightColor->color());
 
   saveColor(UiStyle::Timestamp, ui.timestampFG->color(), ui.timestampBG->color(), ui.timestampUseBG->isChecked());
   s.setValue("timestampUseBG", ui.timestampUseBG->isChecked());
@@ -573,9 +573,9 @@ bool ColorSettingsPage::testHasChanged() {
   if(QtUi::style()->format(UiStyle::RenameMsg).foreground().color() != ui.renameMessageFG->color()) return true;
   if(QtUi::style()->format(UiStyle::RenameMsg).background().color() != ui.renameMessageBG->color()) return true;
   if(settings["RenameMessageUseBG"].toBool() != ui.renameMessageUseBG->isChecked()) return true;
-*/
+
   if(QtUi::style()->highlightColor() != ui.highlightColor->color()) return true;
-/*
+
   if(QtUi::style()->format(UiStyle::Timestamp).foreground().color() != ui.timestampFG->color()) return true;
   if(QtUi::style()->format(UiStyle::Timestamp).background().color() != ui.timestampBG->color()) return true;
   if(settings["TimestampUseBG"].toBool() != ui.timestampUseBG->isChecked()) return true;