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

Review Menu Shown Behind #31144

Closed
metiftikci opened this issue May 28, 2024 · 1 comment · Fixed by #32631
Closed

Review Menu Shown Behind #31144

metiftikci opened this issue May 28, 2024 · 1 comment · Fixed by #32631
Assignees
Labels
topic/ui Change the appearance of the Gitea UI type/bug
Milestone

Comments

@metiftikci
Copy link
Member

metiftikci commented May 28, 2024

Description

after #31062 popup menu shown behind if section collapsed

all headers are sticky and their z-index are same. so upper menu cant visible on front

<h4 class="diff-file-header sticky-2nd-row ui top attached header">

if i decrease z-index of second header it work but i believe this is not the correct solution

Screenshots

image

Edit:

Github uses extra class for menu's header

image

Gitea Version

latest

Can you reproduce the bug on the Gitea demo site?

No

Operating System

No response

Browser Version

latest

@metiftikci metiftikci added topic/ui Change the appearance of the Gitea UI type/bug labels May 28, 2024
@silverwind
Copy link
Member

Ideally such menus shouldn't be z-fighting with the page elements in first place and this could be achieved by using tippy dropdown instead of fomantic dropdown.

@lunny lunny modified the milestones: 1.22.1, 1.23.0 May 29, 2024
@kerwin612 kerwin612 self-assigned this Nov 23, 2024
kerwin612 added a commit to kerwin612/gitea that referenced this issue Nov 25, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Feb 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants