Review Request 117957: kcm_fonts: correctly restore default configuration values

Andrea Iacovitti aiacovitti at libero.it
Sun May 18 01:38:30 BST 2014


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

(Updated May 18, 2014, 12:38 a.m.)


Review request for kde-workspace.


Changes
-------

Changed the combobox connection to currentIndexChanged signal as Thomas correctly suggested, thanks.


Bugs: 324728
    http://bugs.kde.org/show_bug.cgi?id=324728


Repository: kde-workspace


Description
-------

When restoring default configuration:
- Disable "Exclude range checkbox"
- Enable/Disable "Configure..." button accordingly to antialias setting


Diffs (updated)
-----

  kcontrol/fonts/fonts.cpp cf21cb1 

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


Testing
-------

Tested it compiles and works as expected


Thanks,

Andrea Iacovitti

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


More information about the kde-core-devel mailing list