Move exposed form styling into core Olivero theme

+1, either this is going to be an awkward upgrade path or this theme is kept around forever - it feels like the effort spent here would be better off just fixing it in core?

Somewhere/somehow we also have to explain why both "Olivero" and "Olivero for Drupal CMS" show up in the Appearance tab.

The Olivero subtheme isn't only there to work around deficiencies in core. If it were, then I would be in full agreement with you that the energy would be better spent on fixing core.

My understanding is that it's mainly there to provide useful and attractive styling for our content types in their various display modes, until Experience Builder is released and allows us to handle all that without creating a new theme. When XB is released and we've fully adopted it, the subtheme will be deprecated.

It's a shitty compromise, and it does make for some awkwardness, but since XB is not ready, and was not going to be ready in time for Drupal CMS's first stable release, this is probably the best we could do.

@tim.plunkett, @ckrina, and @pameeela could offer more context here.

drupal_cms_olivero.libraries.yml only has:

views:
  css:
    theme:
      css/components/views-exposed-form.css: { }

And there are no templates, or SDCs, or anything else except boilerplate. So unless I'm missing something, it is not doing anything of the things mentioned in #4.

https://git.drupalcode.org/project/drupal_cms_olivero/-/tree/1.x?ref_typ...

It is not doing any of the things mentioned in #4...yet. But it's gonna.

See the related issues...

If those changes are done with CSS only (or maybe even with some other customisations), they could be done in a contrib module. The difference to an Olivero subtheme is if/when that module becomes empty, it can mark itself deprecated, even uninstall itself in an update and mark itself obsolete, whereas a default theme cannot do those things.

I have no idea if the changes are CSS-only, or involve template stuff. That is up to the frontend developers (@mherchel is the main guy). I'm hoping that there will be some way to migrate from a theme to whatever Experience Builder offers, but that's total speculation.

The filters just happened to be the only thing done before RC.

See more WIP:
https://git.drupalcode.org/project/drupal_cms/-/merge_requests/259/diffs
https://git.drupalcode.org/project/drupal_cms/-/merge_requests/287/diffs

And we will definitely ship with components for the XB preview integration.

As to the best way to do this though I have no strong opinion other than there's no way this can be done in core, at least not for now in the timeframe we need it.

Yes some of those things would not be able to go into core because they're for things that don't exist in core as such, but the upgrade path for a sub-theme that's only intended to be temporary is still a problem.

I think a module could do all the things those MRs are doing, it might need a couple of alter hooks on top. But that would allow sites to have Olivero installed + extras, and the module to be deprecated and removed without breaking sites then.

Retitling for the main issue here.