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

The file list is partially visible behind the header #7540

Closed
danxuliu opened this issue Dec 15, 2017 · 1 comment · Fixed by #7663
Closed

The file list is partially visible behind the header #7540

danxuliu opened this issue Dec 15, 2017 · 1 comment · Fixed by #7663
Labels
Milestone

Comments

@danxuliu
Copy link
Member

When the files vertically overflow the list and the list is scrolled its contents are partially visible behind the file list header.

Origin:
files-header-origin

Scrolled:
files-header-scrolled

@nextcloud/designers

@danxuliu danxuliu added bug design Design, UI, UX, etc. feature: files labels Dec 15, 2017
@danxuliu danxuliu added this to the Nextcloud 13 milestone Dec 15, 2017
@jancborchardt
Copy link
Member

It is intended to be slightly visible, however only very slightly with high opacity of the breadcrumb and multiselect bar. Kind of like in iOS - ideally we can use CSS3 blur effect even.

danxuliu added a commit that referenced this issue Jan 2, 2018
Some parts of the file list contents (file name and actions) had a
higher z-index than the file list multiselect header. That header is
fixed in place, so when the file list contents were scrolled and those
parts with a higher z-index overlapped the multiselect header they were
fully visible. Now the z-index for the multiselect header has a higher
value (the same used in the controls bar) to ensure that the contents
are shown behind the header.

Fixes #7540

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
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 a pull request may close this issue.

2 participants