[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [picmi] src/gui: Handle font color selection correctly
From:       Jakob Gruber <jakob.gruber () gmail ! com>
Date:       2015-11-22 10:17:47
Message-ID: E1a0RiN-000472-7R () scm ! kde ! org
[Download RAW message or body]

Git commit f6d598d36b327bd56ca7babd424a7a94c3671f13 by Jakob Gruber.
Committed on 22/11/2015 at 10:16.
Pushed by gruber into branch 'master'.

Handle font color selection correctly

This inverted logic was introduced during porting.

M  +2    -2    src/gui/settingswindow.cpp

http://commits.kde.org/picmi/f6d598d36b327bd56ca7babd424a7a94c3671f13

diff --git a/src/gui/settingswindow.cpp b/src/gui/settingswindow.cpp
index e050e2e..2dd8312 100644
--- a/src/gui/settingswindow.cpp
+++ b/src/gui/settingswindow.cpp
@@ -52,7 +52,7 @@ void SettingsWindow::selectSolvedColor()
 {
     QColor color(m_font_color_solved);
     color = QColorDialog::getColor(color, this);
-    if ( color.isValid() ) {
+    if (!color.isValid()) {
         return;
     }
     m_font_color_solved = color.name();
@@ -63,7 +63,7 @@ void SettingsWindow::selectUnsolvedColor()
 {
     QColor color(m_font_color_unsolved);
     color = QColorDialog::getColor(color, this);
-    if ( color.isValid() ) {
+    if (!color.isValid()) {
         return;
     }
     m_font_color_unsolved = color.name();
[prev in list] [next in list] [prev in thread] [next in thread]