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

AppSettings expands to the bottom when opened #2516

Closed
raimund-schluessler opened this issue Aug 28, 2020 · 6 comments
Closed

AppSettings expands to the bottom when opened #2516

raimund-schluessler opened this issue Aug 28, 2020 · 6 comments
Labels
0. to triage Pending approval or rejection bug

Comments

@raimund-schluessler
Copy link
Member

The AppSettings component expands to the bottom when you open it, instead of showing a scroll bar.

Screenshot_2020-08-28 Woche 36 aus 2020 - Kalender - Nextcloud

Happens on Win10, latest Firefox, Firefox 78.0.1 ESR and latest Chrome. The reason is this line:

overflow-y: visible !important;
which forces the AppSettings content to be completely visible. Replacing it with overflow-x: hidden; (to prevent to long labels from showing a x-scrollbar) prevents the issue.

@raimund-schluessler raimund-schluessler added bug 0. to triage Pending approval or rejection labels Aug 28, 2020
@georgehrke
Copy link
Member

IIRC overflow-x: hidden; caused many other issues. (related to the timezone list i think)

I would say for now this is accepted (although very annoying) and close it in favor of #2304

@raimund-schluessler
Copy link
Member Author

overflow-x: hidden is not really necessary to fix the scrolling. Removing overflow-y: visible would be enough. Is there a reason you force it to be visible?

@georgehrke
Copy link
Member

overflow-x: hidden is not really necessary to fix the scrolling. Removing overflow-y: visible would be enough. Is there a reason you force it to be visible?

BCDFFB62-4123-48EA-A80E-E168A0E3703B

The timezone list is cut off and barely shows 4 entries.

@georgehrke
Copy link
Member

Same for the timeslot size option:
8CED4AC0-9F1D-47BB-9E71-2F2D33C6E353

@raimund-schluessler
Copy link
Member Author

Well, you can scroll to show the rest of the options. But I agree, that is far from perfect.

@raimund-schluessler
Copy link
Member Author

I guess #2304 really is the way to go then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. to triage Pending approval or rejection bug
Projects
None yet
Development

No branches or pull requests

2 participants