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

set overflow to auto #210

Closed
wants to merge 1 commit into from
Closed

set overflow to auto #210

wants to merge 1 commit into from

Conversation

DanielRuf
Copy link

Currently the space for the right scrollbar is always visible.

This is not a great or the right behavior for good webdesign.
It should be set to auto.

@satazor
Copy link
Member

satazor commented Dec 27, 2017

Setting it to auto will make the "dancing" scrollbar effect which is far worse.

@satazor satazor closed this Dec 27, 2017
@DanielRuf
Copy link
Author

Can you show me what you mean? I see no dancing effect in Chrome on Windows 10.

Then it is a problem with some other property in combination with overflow. The right empty scrollbar should not always be there.

@satazor
Copy link
Member

satazor commented Dec 27, 2017

With auto, the homepage will have no scrollbar but once you search something it will have a scrollbar. Because scrollbars take space on widows and since we have smooth transitions between pages this causes a ugly shift of the site contents and visuals.

@DanielRuf
Copy link
Author

I saw no shift. I'll try some other approach for example using calculated widths, will-change and translateZ.

Setting it generally to scroll adds always the scrollbar container.

@DanielRuf
Copy link
Author

Setting it to scroll is not the best and real solution for this imo.

@satazor
Copy link
Member

satazor commented Dec 27, 2017

Are you using a surface or a touch device? This happens on desktop using a mouse. I don’t have time to make a video specially because I don’t use windows, but I will try to find a video.

@DanielRuf
Copy link
Author

No, desktop / laptop.
I'll try to reproduce the described behavior and find the best solution. At least I never had to use the scroll value and fix it wit better approaches eg by using separate scroll containers, styled / hidden scrollbars and precalculating the changes.

@DanielRuf DanielRuf deleted the patch/body-overflow branch April 21, 2018 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants