🇬🇷Greece @iro

vensires → credited iro → .

For future reference, the test results in a glance:

I tested the cases described at #6 and they work as described. I tested all combinations.

There are unresolved threads at the MR so I am reverting this issue to needs work.

The should only be one visible header at any time.
At the given screenshots it is clear that this is not the case.

While testing on Firefox 130.0.1 on linux, I noticed that there are alignment issues on all screen sizes.
On chrome alignment is ok.

The alignment is different because the height of the input is different.
The problem seems to be caused by css that styles#admin-toolbar-search-input.
The input inherits height: 100%; from admin.toolbar_search.css in admin_toolbar module which is interpreted differently in the two browsers.
If it gets overwritten in gin theme the problem if fixed (either by height: 30px of max-height: 30px).
It can be done in toolbar.scss in gin theme since the id styles overwrites are placed there.

lauriii → credited iro → .

kostask → credited iro → .

I tested on Drupal 10.2.5 and I cannot reproduce.
I added to an entity a term with four level depth using cshs. Edited and saved again with less depth and it works as expected.

I tested on a clean installation of drupal 10.2.5 and I cannot reproduce the issue:

I tested the current implementation. I have limited knowledge of the current state of the designs, but the implementation seems consistent:
I am uploading 3 screenshots from the current state:
- Menu level 4 not active without hover
.
- Menu level 4 not active with hover
.
-Menu level 4 active with hover
.

Patch #2 works for me too.

iro → created an issue.

iro → created an issue.

Patch applies and works as expected on Drupal 9.5.8.