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

Dashboard main container size gets changed #19

Open
bfbones opened this issue Feb 9, 2021 · 2 comments
Open

Dashboard main container size gets changed #19

bfbones opened this issue Feb 9, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@bfbones
Copy link

bfbones commented Feb 9, 2021

Environment

Nextcloud 20.0.4
Vivaldi 3.5 / Fedora

Description

Using this addon the dashboard container size gets changed and its background accordingly. Content is missing in height and the design seems broken.

Screenshot

Screenshot_2021-02-09_14-43-40

Affected code

# css/dashboard.css
body {
        height: auto;
        min-height: 100%;
}

If this code gets simply removed it works just fine.
This problem also exists for https://github.com/nextcloud/integration_gitlab

@julien-nc
Copy link
Member

Thanks for the issue. This piece of css is a an attempt to fix the incorrect position of items context menus when widgets are not on the first row.

I can remove that and wait for a fix in nc/vue.
Or do you have a magic css trick to fix both problems?

This context menu topic is discussed in nextcloud-libraries/nextcloud-vue#1384

@bfbones
Copy link
Author

bfbones commented Feb 9, 2021

So then it's good, that I dont simply opened a MR.
I did not enough testing to bring any magic css tricks for now, so maybe quickfix it first and wait for nc/vue.

@MB-Finski MB-Finski added the bug Something isn't working label Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants