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

List:       kde-commits
Subject:    branches/KDE/3.5/kdegraphics/kolourpaint/widgets (silent)
From:       Clarence Dang <dang () kde ! org>
Date:       2007-10-07 8:03:50
Message-ID: 1191744230.911228.21527.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 722398 by dang:

SVN_SILENT KColorDialog comment from trunk/


 M  +11 -0     kpcolortoolbar.cpp  


--- branches/KDE/3.5/kdegraphics/kolourpaint/widgets/kpcolortoolbar.cpp #722397:722398
@@ -245,6 +245,17 @@
         QColor col = Qt::black;
         if (color (whichColor).isOpaque ())
             col = color (whichColor).toQColor ();
+        else
+        {
+            // TODO: If you double-click on a transparent color and press OK, you get
+            //       black, instead of the color staying as transparent.
+            //
+            //       We should modify or fork KColorDialog to allow us to fix this.
+            //
+            //       It would be wrong to stop the user from double-clicking on a
+            //       transparent color as that would make the UI inconsistent, compared
+            //       to opaque colors.
+        }
 
         // TODO: parent
         if (KColorDialog::getColor (col/*ref*/))
[prev in list] [next in list] [prev in thread] [next in thread]