Review Request: Use platform palette and fonts when running on other desktop environments

[Aurélien Gâteau]

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

(Updated July 2, 2011, 9:19 p.m.)


Review request for kdelibs and Olivier Goffart.


Changes
-------

Updated to check whether we run in a KDE session. Applied the same change to font settings.


Summary (updated)
-------

When a KDE application is running on GNOME it looks odd right now because it does not use the GNOME palette and fonts, contrary to Qt-only applications. Attached patch fixes this by relying on the platform plugin to set the correct palette and fonts if we are not running in a full KDE session.

Patch was suggested by Olivier Goffart.


Diffs (updated)
-----

  kdeui/kernel/kglobalsettings.cpp 1a497c7 

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


Testing (updated)
-------

# On KDE
- Run kwrite on KDE => KDE palette and fonts
- Change palette and fonts from System Settings => kwrite updates itself correctly

# On GNOME
- Run kwrite on GNOME => GNOME palette and fonts
- Change palette and fonts from GNOME Tweak Tool => palette gets applied, font does not for now


Thanks,

Aurélien

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