expose the colorset for the stylesheet
ClosedPublic

Authored by mart on Sep 20 2017, 4:58 PM.

Details

Summary

this makes possible having icons loaded with different
colors in different parts of the application, will be needed
by kirigami (which will use this with a plasma-integration extension)

Test Plan

i can have icons colored in different ways across a single app

Diff Detail

Repository
R302 KIconThemes
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mart created this revision.Sep 20 2017, 4:58 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptSep 20 2017, 4:58 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
mart planned changes to this revision.Sep 20 2017, 4:59 PM
mart updated this revision to Diff 19787.Sep 22 2017, 1:07 PM
  • make it work
mart edited the test plan for this revision. (Show Details)Sep 22 2017, 1:12 PM
mart added reviewers: Frameworks, Plasma, hein.
mart added a reviewer: dfaure.
dfaure accepted this revision.Sep 24 2017, 8:49 AM
dfaure added inline comments.
src/kiconloader.h
465

I guess this is OK as is, so remove the comment (it's not we can change this later)

This revision is now accepted and ready to land.Sep 24 2017, 8:49 AM
mart updated this revision to Diff 19935.Sep 26 2017, 12:27 PM
  • remove comment
Restricted Application added a project: Plasma. · View Herald TranscriptSep 26 2017, 12:27 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
This revision was automatically updated to reflect the committed changes.
mart added inline comments.Sep 29 2017, 2:41 PM
src/kiconloader.h
465

apparently this is giving problems indeed, polkit is now failing to build since it wasn't explicitly linking to KConfigWidgets, so may be necessary to duplicate the enum

Since this change, polkit-kde-agent-1 fails to build from source with the error:

In file included from /usr/include/KF5/KIconThemes/KIconLoader:1:0,
from /<<BUILDDIR>>/polkit-kde-agent-1-5.10.95+p17.10+git20170929.1419/AuthDialog.cpp:37:
/usr/include/KF5/KIconThemes/kiconloader.h:31:10: fatal error: kcolorscheme.h: No such file or directory
#include <kcolorscheme.h>
^~~~~~~~~~~~~~~~
compilation terminated.
mart added a comment.Sep 29 2017, 6:02 PM

D8060 solves the same problem without new dependencies