Branch/Tag selector is scrollable #5537

Closed
opened 2025-11-02 06:28:15 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @UnlimitedCookies on GitHub (Jun 12, 2020).

Reproducible on any repository on https://gitea.com/ and https://try.gitea.io/

Description

There is a scroll bar on the branch/tag selector and you can scroll down.
When you scroll down the text is not readable.

Screenshots

Anmerkung 2020-06-12 125040

This is how it looks when scrolled down:
Anmerkung 2020-06-12 125105

Originally created by @UnlimitedCookies on GitHub (Jun 12, 2020). Reproducible on any repository on https://gitea.com/ and https://try.gitea.io/ ## Description There is a scroll bar on the branch/tag selector and you can scroll down. When you scroll down the text is not readable. ## Screenshots ![Anmerkung 2020-06-12 125040](https://user-images.githubusercontent.com/49200216/84495647-cb64fe00-acab-11ea-8c4e-c3ec652ff0b4.png) **This is how it looks when scrolled down:** ![Anmerkung 2020-06-12 125105](https://user-images.githubusercontent.com/49200216/84495650-cdc75800-acab-11ea-9d92-027255aec5b3.png)
Author
Owner

@CirnoT commented on GitHub (Jun 12, 2020):

Caused by

.dropdown:not(.selection)>.menu:not(.review-box)>* {
    overflow-x: hidden;
}
@CirnoT commented on GitHub (Jun 12, 2020): Caused by ```css .dropdown:not(.selection)>.menu:not(.review-box)>* { overflow-x: hidden; } ```
Author
Owner

@CirnoT commented on GitHub (Jun 12, 2020):

This might fix it:

- .dropdown:not(.selection) > .menu:not(.review-box) > *
+ .dropdown:not(.selection) > .menu:not(.review-box) > *:not(.header)
@CirnoT commented on GitHub (Jun 12, 2020): This might fix it: ```diff - .dropdown:not(.selection) > .menu:not(.review-box) > * + .dropdown:not(.selection) > .menu:not(.review-box) > *:not(.header) ```
Author
Owner

@silverwind commented on GitHub (Jun 12, 2020):

I think it's time we remove the .dropdown:not(.selection) > .menu:not(.review-box) > * abomination. It was aimed at fixing dropdown menu overflow but it's causing just too many issues also seen by those :not's.

@silverwind commented on GitHub (Jun 12, 2020): I think it's time we remove the `.dropdown:not(.selection) > .menu:not(.review-box) > *` abomination. It was aimed at fixing dropdown menu overflow but it's causing just too many issues also seen by those `:not`'s.
Author
Owner

@CirnoT commented on GitHub (Jun 12, 2020):

Then we'll be back at original issue with scrollbars where unnecessary on dropdowns. We could maybe try applying the fix and see if anything else comes up being broken, before deciding to abandon this approach entirely?

@CirnoT commented on GitHub (Jun 12, 2020): Then we'll be back at original issue with scrollbars where unnecessary on dropdowns. We could maybe try applying the fix and see if anything else comes up being broken, before deciding to abandon this approach entirely?
Author
Owner

@silverwind commented on GitHub (Jun 12, 2020):

Yeah I guess I won't block a fix like above. Just wanted to mention that it's getting a bit out of hand in terms of selector complexity 😉

@silverwind commented on GitHub (Jun 12, 2020): Yeah I guess I won't block a fix like above. Just wanted to mention that it's getting a bit out of hand in terms of selector complexity 😉
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#5537