Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump-targets have higher z-index than dropdown-menu #6746

Closed
zoglo opened this issue Jan 19, 2024 · 0 comments · Fixed by #6747
Closed

Jump-targets have higher z-index than dropdown-menu #6746

zoglo opened this issue Jan 19, 2024 · 0 comments · Fixed by #6747
Labels
Milestone

Comments

@zoglo
Copy link
Contributor

zoglo commented Jan 19, 2024

Affected version(s)

5.3.0-RC1

Description

Bug

  • The z-index of the jump-targets (3) is higher than the user-menu (2), hence overlapping

image

This does not occur sub 991px due to #main using will-change: transform (See Stacking-context)

Having a z-index: 1 on the jump-targets should be sufficient.

@leofeyer leofeyer added this to the 5.3 milestone Jan 19, 2024
@leofeyer leofeyer linked a pull request Jan 19, 2024 that will close this issue
leofeyer added a commit that referenced this issue Jan 25, 2024
Description
-----------

Fixes #6746 

A z-index of 1 should be sufficient for `.jump-targets`, we could also raise the z-index of the dropdown-menu instead.

Commits
-------

e200f00 Adjust `z-index` for `.jump-targets`
16785ea Rebuild the assets
3a72ff3 Raise `z-index` on dropdown-menu
738329c Increase z-index for `tl_formbody_submit` and add comments
44332ff Rebuild the assets
c217404 Rebuild the assets
6e5527c Adjust the z-index of the .tl_formbody_submit rule

Co-authored-by: leofeyer <1192057+leofeyer@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants