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

Backend sidebar sticky on scroll #496

Closed
sbonardt opened this issue Jun 28, 2019 · 1 comment · Fixed by #542
Closed

Backend sidebar sticky on scroll #496

sbonardt opened this issue Jun 28, 2019 · 1 comment · Fixed by #542

Comments

@sbonardt
Copy link
Contributor

For the new responsive layout, we need the backend sidebar with the menu sticky scrollable. For niceness

@sbonardt
Copy link
Contributor Author

It seems adding the supported position: sticky results in sidebar height problems. That is, the blue background of the sidebar does not extend fully on not very high viewports (when testing the sticky). So the sticky bit works but because of the overflowing submenus on the main menu on not very high viewports the overflowing forces extra whitespace after the sidebar which does not extend because we are positioning sticky...

I advise not to use the position sticky for now. Also the result was a bit weird. It did position sticky itself and upon scrolling everything worked as should be, but it felt kind of strange. I can imagine this feeling extra strange for users on a backend with a very long (high) main menu in the sidebar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants