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

Add a z-index to the limit toggler #6661

Merged
merged 1 commit into from Dec 27, 2023
Merged

Conversation

zoglo
Copy link
Contributor

@zoglo zoglo commented Dec 25, 2023

Fixes #6660

  • Adds a z-index to the limit-toggler to make it clickable again

@leofeyer leofeyer added this to the 5.3 milestone Dec 27, 2023
@leofeyer leofeyer changed the title Add z-index to limit-toggler (#6660) Add z-index to limit-toggler Dec 27, 2023
@leofeyer
Copy link
Member

How can I reproduce the problem e.g. with the official demo?

@de-es
Copy link
Contributor

de-es commented Dec 27, 2023

Das Problem tritt aktuell nur in 5.x auf, nicht in 5.2. Daher in der offiziellen Demo nicht nachstellbar. Hatte es auch schon bemerkt, aber @zoglo war schneller. 👍

@leofeyer
Copy link
Member

Bei mir tritt das Problem auch in der neuesten 5.x nicht auf, daher die Frage, wie ich es in der Official Demo (nicht der Online-Demo) reproduzieren kann.

@zoglo
Copy link
Contributor Author

zoglo commented Dec 27, 2023

@leofeyer
I am also using the contao-demo for testing and it happens using Chrome, as well as Firefox on Windows when trying to click the '...' (limit-toggler) of a content-element.

Navigate into Articles -> Edit the content elements of an article (e. g. Content-Elements) -> try to click any toggler
image

In the following screenshot (I resized the window), it's even hidden behind the table:
image

@leofeyer leofeyer merged commit e31cb1f into contao:5.x Dec 27, 2023
17 checks passed
@leofeyer
Copy link
Member

Thank you @zoglo.

@zoglo zoglo deleted the bugfix/limit_toggler branch December 27, 2023 15:00
@leofeyer leofeyer changed the title Add z-index to limit-toggler Add a z-index to the limit toggler Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

limit_toggler button (...) does not work anymore
4 participants