the administration theme option does not check for theme compatibility.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | admin_theme_core_compatibility-1013268-4.patch | 2.18 KB | jrreid |
| #3 | admin_theme_core_compatibility-1013268-3.patch | 2.3 KB | pillarsdotnet |
| #1 | admin_theme.patch | 1.86 KB | droplet |
Comments
Comment #1
droplet commentedIt only check disabled theme for better performance. If we changed theme compatibility after its enabled, it won't check too. My patch is check all themes and disable theme not compatible with D7 (but not really alter settings in DB, only interface changes)
Comment #2
droplet commentedfirst in D8 first and backport
Comment #3
pillarsdotnet commentedRe-roll using
git format-patch. No change necessary, as the patch still applies cleanly.Comment #4
rstamm commentedit would be nice if the compatibility check moves into a helper function.
contrib modules can then also use it.
Comment #5
jrreid commentedRerolling for 8.x
Comment #6
kscheirer#5: admin_theme_core_compatibility-1013268-4.patch queued for re-testing.
Comment #7
kscheirerRetesting against latest HEAD since it has been over a year.
Comment #18
pameeela commentedNot clear what needs to be done here. Can anyone update the issue summary with some more info?
Comment #21
quietone commentedYes, it is not clear what needs to be done here and there has been no reply.
Furthermore, As of Drupal 8.7.7 themes declare compatibility with core. See New 'core_version_requirement' key in info.yml files for modules, themes and profiles allows Composer semantic version constraints including specifying multiple major versions of core.
I think this can be closed as outdated.