Review Request 112279: Call the correct static Q[Gui]Application::setFont

Commit Hook null at kde.org
Mon Aug 26 14:27:36 UTC 2013


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

(Updated Aug. 26, 2013, 2:27 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Kevin Ottens.


Description
-------

When fonts changes, call the correct static depending whether the
QCoreApplication::instance is a QApplication or QGuiApplication.
    
The reason why QGuiApplication is not enough is because we need to
propagate/send a QEvent to each Widget to make them aware that the
font has changed and QGuiApplication know nothing about widgets


Diffs
-----

  staging/frameworkintegration/src/platformtheme/kfontsettingsdata.cpp 70c1d26 

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


Testing
-------

Tested fontChange with QApplication and QGuiApplication


Thanks,

Àlex Fiestas

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


More information about the Kde-frameworks-devel mailing list