Review Request 125303: [Cantor] Drop KGlobalSettings and implements a substitute to KGlobalSettings::kdisplayPaletteChanged

Jeremy Whiting jpwhiting at kde.org
Fri Sep 18 20:08:36 UTC 2015


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


This looks good, but couldn't you simply connect the slot to the QApplication paletteChanged signal instead of adding an event filter? http://doc.qt.io/qt-5/qguiapplication.html#paletteChanged

- Jeremy Whiting


On Sept. 18, 2015, 1:47 p.m., Filipe Saraiva wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125303/
> -----------------------------------------------------------
> 
> (Updated Sept. 18, 2015, 1:47 p.m.)
> 
> 
> 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/43ac29b3/attachment.html>


More information about the kde-edu mailing list