Problem/Motivation
#3277057: Make Claro the default admin theme in Standard profile added block configuration for Claro to Standard install profile. However, Claro already ships with block configuration in the theme itself.
Proposed resolution
Since Claro already ships with its block configuration, we should be able to remove the block configuration from Standard.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3278565-2.patch | 5.02 KB | lauriii |
Comments
Comment #2
lauriiiComment #3
lauriiiComment #5
alexpottClaro doesn't ship with the config anymore it needs to be moved back in 9.4.x/9.5.x/10.0.x as far as I know,
Comment #6
lauriiiI think the block config is still there: https://git.drupalcode.org/project/drupal/-/tree/9.4.x/core/themes/claro... , https://git.drupalcode.org/project/drupal/-/tree/9.5.x/core/themes/claro... and https://git.drupalcode.org/project/drupal/-/tree/10.0.x/core/themes/clar...
Comment #7
spokjeAgreeing with @lauriii here, block config for Claro is still present in
9.4.x-dev,9.5.x-devand10.0.x-devas shown by the links in #6.I _think_ @alexpott is thinking about the Olivero block config here, which is(n't)/was(n't)/could be present in [insert-random-branch]. (Yes, lost track myself).
This one however is RTBC for me.
Comment #9
catchCommitted/pushed to 10.0.x, cherry-picked to 9.5.x and 9.4.x.
#3278674: Decide how blocks should be shipped for Claro and Olivero is open for deciding exactly what we want to do long term here, but getting to a consistent and working (and potentially final if we decide it's what we want after all) is great.