Problem/Motivation

The z-index of the app-header is VERY high:

.app-header {
	z-index: 1034;
	grid-area: lte-app-header;
	max-width: 100vw;
	border-bottom: 1px solid var(--bs-border-color);
	transition: 0.3s ease-in-out;
}

Admin Toolbar Search has:

&.ui-autocomplete {
	z-index: 600;
}

So the header overlays the admin element and I don't think Admin Toolbar Search wll count even higher.

Is there a reason for that super high value? I guess in Drupal it should be overridden with a much lower value, maybe looking into other Drupal Themes?

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork adminlte-3608932

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

anybody created an issue. See original summary.

anybody’s picture

Title: z-index of .app-header overlays Admin Toolbar Search » z-index of .app-header, sidebar, ... overlays Admin Toolbar Search

Okay it's the same issue for the sidebar!

anybody’s picture

Assigned: Unassigned » thomas.frobieter
anybody’s picture

Version: 1.0.0-beta1 » 1.0.x-dev
Assigned: thomas.frobieter » Unassigned
Status: Active » Fixed

This is fixed now, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

thomas.frobieter’s picture

Assigned: Unassigned » thomas.frobieter
Status: Fixed » Needs work

No, this was just fixed locally.

thomas.frobieter’s picture

Status: Needs work » Needs review

Drupals .toolbar-tray has a z-index of 501. So we need to have lower z-index for .app-header and .app-sidebar.

thomas.frobieter’s picture

Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

thomas.frobieter’s picture

Status: Fixed » Needs work

Oh wait, my fault. It already was 20 + 30, which is fine, but both are overridden by the adminlte.css which sets much higher z-indexes.

On it..

thomas.frobieter’s picture

Assigned: thomas.frobieter » Unassigned
Status: Needs work » Needs review

Okay, I've moved the Drupal-specific styles and scripts to their own library. Now, we can use library dependencies to fix this.
Another option would be to use !important, but the Drupal tweaks are generally ment to override adminlte default styles.

anybody’s picture

Status: Needs review » Reviewed & tested by the community

anybody’s picture

Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.