Refactor components

it is ok in terms of BEM:

level - modifier name

2 - modifier value

TIL that you have to do that separation! :D https://en.bem.info/methodology/naming-convention/#two-dashes-style

I've just left a few comments about comments, so moving to NW just for that. Basically, the idea behind those comments is that any newbie (or just somebody without enough context) landing on that file in the future can understand what each part is doing. We don't have external documentation, so we rely on the fields themself to be the documentation itself. Olivero's navigation JS file is a great example: https://git.drupalcode.org/project/drupal/-/blob/11.x/core/themes/oliver...

About the rest, everything makes a lot of sense to me: I personally don't see anything I'd like to change. But it would be great if somebody else could take a look to this, so I'll ping a few people in Slack to see if they can give a quick look to this.

Thanks for the proposal, looks great!

I added few comments and simplified js.
I think we may need dedicated ticket about js documentation according core standards.

This looks good to go for me since it's a net improvement, thanks @finnsky! I'll give some time before merging to see if somebody else finds something worth being improved.

OK, time to merge this. Thanks for the work @finnsky!

Automatically closed - issue fixed for 2 weeks with no activity.