Review Request 127536: KNewPasswordWidget: fix QPalette when the widget is disabled

Elvis Angelaccio elvis.angelaccio at kdemail.net
Mon Apr 4 14:33:39 UTC 2016


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

(Updated April 4, 2016, 2:33 p.m.)


Review request for KDE Frameworks, Aleix Pol Gonzalez, Christoph Feck, and David Faure.


Changes
-------

The bug also affects to the default QPalette of the verification line edit, when the widget is disabled before `init()` is executed.


Repository: kwidgetsaddons


Description
-------

If the widget gets disabled, the verification's QLineEdit is disabled as well, but its QPalette is wrong (see the screenshots).
This patch fixes the bug by setting the Active color group for the palette.


Diffs (updated)
-----

  autotests/knewpasswordwidgettest.h 3b7418700557cd3a2b3de91127f76a5bf99345d3 
  autotests/knewpasswordwidgettest.cpp 4507dbdc899407afb72ffa14437f050aae0b57df 
  src/knewpasswordwidget.cpp 72748d6b9e9fed979bd75ce856867a7bbf9e2fbd 

Diff: https://git.reviewboard.kde.org/r/127536/diff/


Testing
-------

A test case is added to reproduce the bug.


File Attachments
----------------

Before
  https://git.reviewboard.kde.org/media/uploaded/files/2016/03/31/d4d0d5be-91d4-4787-adc2-032d358a3dc5__knewpasswordwidget-disabled1.png
After
  https://git.reviewboard.kde.org/media/uploaded/files/2016/03/31/46e24106-ceb5-486f-8222-d928c8690929__knewpasswordwidget-disabled2.png


Thanks,

Elvis Angelaccio

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160404/d5e30dfa/attachment.html>


More information about the Kde-frameworks-devel mailing list