Skip to content

refactor: Dropped themes decorator for explicit initialization#1746

Merged
simeonoff merged 8 commits intomasterfrom
rkaraivanov/theme-controller-migration
Jun 25, 2025
Merged

refactor: Dropped themes decorator for explicit initialization#1746
simeonoff merged 8 commits intomasterfrom
rkaraivanov/theme-controller-migration

Conversation

@rkaraivanov
Copy link
Copy Markdown
Member

Migrated components to directly use the themes controller instead of the old decorator. Aside from being more explicit about it, this would help untangle some inheritance chain duplication as well as possibly implmenting lazy-loading of additional styles at runtime, potentially reducing the initial payload size greatly.

Migrated components to directly use the themes controller
instead of the old decorator. Aside from being more explicit
about it, this would help untangle some inheritance chain
duplication as well as possibly implmenting lazy-loading
of additional styles at runtime, potentially reducing
the initial payload size greatly.
@rkaraivanov rkaraivanov marked this pull request as ready for review June 19, 2025 16:16
@simeonoff simeonoff merged commit db68dd5 into master Jun 25, 2025
5 checks passed
@simeonoff simeonoff deleted the rkaraivanov/theme-controller-migration branch June 25, 2025 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants