react-custom-scrollbars icon indicating copy to clipboard operation
react-custom-scrollbars copied to clipboard

fixed issue #50 when native scrollbars are visible when zoomed out. p…

Open ghost opened this issue 7 years ago • 7 comments

  • [x] fixed issue #50 when native scrollbars are visible when zoomed out.
  • [x] prevented infinite loop in the colored example

ghost avatar Oct 04 '16 17:10 ghost

checking scrollbar width on every handleWindowResize would lead to a huge performance drop and since this is an edge case I'm not really happy with this approach

malte-wessel avatar Oct 05 '16 17:10 malte-wessel

Unfortunately there is no way to get 'zoom' event. Browser's zoom triggers resize event. Window resizing is quite rare event and on resize user doesn't scroll anything, so it shouldn't have a drop performance affect

ghost avatar Oct 05 '16 22:10 ghost

I'm still having this issue. Would it make sense to throttle/debounce this to prevent a too huge performance drop?

Ponjimon avatar Dec 12 '16 14:12 Ponjimon

@malte-wessel I understand why you're not happy with this implementation, but window resize is still an issue with this package.

Do you think we could, at least, have a public method (this.refs.scrollbar.forceUpdate() ?) to force width recalculation ? This way the package doesn't update by default (no performance drop), but user still have a possibility to trigger recalculation by itself.

I could create a Merge Request if you like the idea

Colmea avatar Apr 18 '18 12:04 Colmea

This is a problem for us as well. Is this library dead? :-)

sorenhoyer avatar May 09 '18 19:05 sorenhoyer

@sorenhoyer looks like it's dead) the problem exists for a long time and there are no solution for it((

DeRoPega avatar Sep 21 '18 14:09 DeRoPega

You could use react-custom-scrollbars-2 will fix this issue

hoanglvbd avatar Jan 25 '22 03:01 hoanglvbd