Review Request 115285: Pass the KDE_COLOR_SCHEME_PATH application property to toplevel windows

Martin Gräßlin mgraesslin at kde.org
Fri Jan 24 09:36:00 UTC 2014


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

Review request for KDE Frameworks and Hugo Pereira Da Costa.


Repository: frameworkintegration


Description
-------

Pass the KDE_COLOR_SCHEME_PATH application property to toplevel windows

This feature is upstreamed from the Oxygen style. It is a general
feature for integration with our window manager and thus should be in
the generic style implementation and not just in Oxygen.

So far the implementation only supports X11 and installs an X property
on all toplevel windows. On other platforms it is currently a no-op.

This introduces an optional dependency on XCB and QtX11Extras. If they
are not found the feature is obviously not compiled.


Diffs
-----

  src/kstyle/CMakeLists.txt 41c92c266a92e4e3534114c982ed98cf529456d5 
  src/kstyle/kstyle.cpp f79675b85a58d0f132bc3d336996d51a2661e567 

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


Testing
-------

Tested with the kcolorschemedemo in kconfigwidgets/tests and an adjusted Oxygen without the feature.


Thanks,

Martin Gräßlin

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


More information about the Kde-frameworks-devel mailing list