npms-io / npms-www

The https://npms.io website
MIT License
245 stars 35 forks source link

set overflow to auto #210

Closed DanielRuf closed 6 years ago

DanielRuf commented 6 years ago

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 commented 6 years ago

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

DanielRuf commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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

satazor commented 6 years ago

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 commented 6 years ago

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.