Problem/Motivation
#2228093: Modernize theme initialization introduced the concept of ThemeManager, a service used to determine/set the "active" theme, render and to alter. At the time, this was needed because "nothing else really existed" ("Hey ThemeHandler...").
Because of this addition, it also made it quite confusing to remember that ThemeManger alters, but ThemeHandler does not (which is different from ModuleHandler).
Proposed resolution
All the current methods of ThemeManager should be proxied to the theme plugin manager or respective @Theme plugin and the equivilent ThemeManager method deprecated.
Remaining tasks
- Create patch
- Create tests (is this even needed, we should already have tests, yes?)
User interface changes
None
API changes
Deprecates ThemeManager in favor of the theme plugin mananger.
Data model changes
None
Comments
Comment #2
markhalliwellComment #3
markhalliwellComment #4
markhalliwellComment #16
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!