Review Request 108637: Fix KColorButton change color + cancel behaviour

Albert Astals Cid aacid at kde.org
Tue Feb 5 23:53:42 GMT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108637/
-----------------------------------------------------------

(Updated Feb. 5, 2013, 11:53 p.m.)


Review request for kdelibs and Laurent Montel.


Description (updated)
-------

Do not connect colorSelected to _k_colorChosen otherwise two bad things happen:
 a) Selecting a color and canceling still changes the color of the color button (this is KColorButtonTest::testChangeAndCancel that fails without my change)
 b) If you are in Recent Colors there's a crash (bug 313984 because a lot of bad things happen (this is KColorButtonTest::testRecentColorsPick that crashes without my change)

Laurent added
     connect(dialog, SIGNAL(colorSelected(QColor)), q, SLOT(_k_colorChosen()));
to make sure that double click worked, thus i've added KColorButtonTest::testDoubleClickChange() to prove it still works after my change

Comments?

Developed against master, plan to apply against KDE/4.10 too


Diffs
-----

  kdeui/colors/kcolorbutton.cpp 786cb9d 
  kdeui/tests/CMakeLists.txt 85f12ed 
  kdeui/tests/kcolorbuttontest.h PRE-CREATION 
  kdeui/tests/kcolorbuttontest.cpp PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/108637/diff/


Testing
-------

Okular no longer crashes, the tests


Thanks,

Albert Astals Cid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130205/18e111d2/attachment.htm>


More information about the kde-core-devel mailing list