ColorSet concept in Theme
ClosedPublic

Authored by mart on Sep 22 2017, 4:47 PM.

Details

Summary

Theme is now an attached property rather than a singleton (in 2.2 import)
a "colorSet" property can be defined on it and all its color properties
will change to that given set (which follows KColorScheme ColorSet)
all child items will inherit the theme of ancestor objects, until instructed not to

Test Plan

Kirigami Gallery

Diff Detail

Repository
R169 Kirigami
Branch
arcpatch-D7939
Lint
No Linters Available
Unit
No Unit Test Coverage
mart created this revision.Sep 22 2017, 4:47 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptSep 22 2017, 4:47 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart added a reviewer: hein.Sep 22 2017, 4:54 PM
mart updated this revision to Diff 20005.Sep 27 2017, 4:48 PM
  • Merge branch 'master' into mart/colorcontext
  • add a rangeslider to the gallery
  • some comments
mart updated this revision to Diff 20006.Sep 27 2017, 4:59 PM
  • remove binary file
  • give focus on press by default
mart updated this revision to Diff 20106.Sep 29 2017, 5:20 PM
  • bind missing colors
  • add missing roles
  • complete colors definition for fallback qml themes
  • active coloring when pressed
  • Merge branch 'master' into mart/colorcontext
  • support static linking with included pri and qmake
  • better docs
  • package extra icons
  • use the Material ripple for list items
  • PlatformThemeFactory->KirigamiPluginFactory
  • finish rename files
  • replace deprecated values
  • all colors in the smallest fallback Theme.qml
  • more documentation
mart updated this revision to Diff 20110.Sep 29 2017, 6:18 PM
  • support for custom colors
  • support "color" property for icons
mart updated this revision to Diff 20247.Oct 2 2017, 1:50 PM
  • finalize signatures for signals
  • todo on font change
hein accepted this revision.Oct 3 2017, 2:33 PM
This revision is now accepted and ready to land.Oct 3 2017, 2:33 PM
mart updated this revision to Diff 20297.Oct 3 2017, 2:44 PM
  • Merge branch 'master' into arcpatch-D7939
This revision was automatically updated to reflect the committed changes.
ratijas added inline comments.
src/libkirigami/platformtheme.h
224–225 ↗(On Diff #20247)

What was this TODO all about?

meven added a subscriber: meven.Jun 15 2023, 7:40 AM
meven added inline comments.
src/libkirigami/platformtheme.h
224–225 ↗(On Diff #20247)

I am guessing adding making colorsChanged(Kirigami::PlatformTheme::ColorSet colorSet).