Review Request 125303: [Cantor] Drop KGlobalSettings and implements a substitute to KGlobalSettings::kdisplayPaletteChanged
Filipe Saraiva
filipe at kde.org
Fri Sep 18 19:47:24 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125303/
-----------------------------------------------------------
Review request for KDE Edu, Aleix Pol Gonzalez, Alexander Rieder, and Michal Humpula.
Repository: cantor
Description
-------
Hi devs, I am finishing the drop of KDELibs4Support from Cantor. I am removing KGlobalSettings now, but I need to port kdisplayPaletteChanged signal. I wrote this patch based on this accepted review request [114619](https://git.reviewboard.kde.org/r/114619/) but I don't know if it is correct.
Well, just say me if it is ok for the moment. I don't know how to test specifically this feature.
Diffs
-----
src/lib/defaulthighlighter.h e33256a
src/lib/defaulthighlighter.cpp 45c7063
src/worksheettextitem.cpp 9fba297
Diff: https://git.reviewboard.kde.org/r/125303/diff/
Testing
-------
* Build ok;
* Installation ok;
* I used Cantor for some time, looks like ok.
Thanks,
Filipe Saraiva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150918/80a92f17/attachment.html>
More information about the kde-edu
mailing list