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

Double scroll bar with long activity list #23406

Closed
nickvergessen opened this issue Oct 13, 2020 · 3 comments
Closed

Double scroll bar with long activity list #23406

nickvergessen opened this issue Oct 13, 2020 · 3 comments
Labels

Comments

@nickvergessen
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Have a file
  2. Change it regularly and often
  3. Scroll the list

Expected behaviour

There is 1 scroll bar

Actual behaviour

There is a double scroll bar

Bildschirmfoto von 2020-10-13 09-29-12

Server configuration

Nextcloud version: 20.0.0

Currently the talk app is already hacking the sidebar to circumvent this and I think it's time to fix this in the files app instead. all sidebar tabs should be height limited and then scroll only there and the file list should have its scroll bar next to it and not disconnected after a sidebar.

cc @danxuliu @skjnldsv @juliushaertl

@nickvergessen nickvergessen added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap feature: activity and notification feature: file sidebar Related to the file sidebar component feature: files and removed feature: files labels Oct 13, 2020
@skjnldsv
Copy link
Member

Expected behaviour

There is 1 scroll bar

Actual behaviour

There is a double scroll bar

Well, there are two elements that can scroll, there will always be two scrollbars.
How else do you want it to be? Not scollable in the AppSidebar? How do we display more items?

@skjnldsv
Copy link
Member

skjnldsv commented Oct 13, 2020

and the file list should have its scroll bar next to it and not disconnected after a sidebar.

This will not happen.
The whole nextcloud is scrolling, not just the inner files. This created tons of issues before.
That is how all websites are doing it, let's stop hacking and use standard html/css implementations.

reference #9982

@skjnldsv
Copy link
Member

Capture d’écran_2020-10-17_16-58-15
Google example.

@skjnldsv skjnldsv added invalid and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Oct 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants