Skip to content

Add support for custom scrollbars in Grid and MultiGrid - #1532

Closed
fazouane-marouane wants to merge 10 commits into
bvaughn:masterfrom
fazouane-marouane:mar/scrollable
Closed

fazouane-marouane wants to merge 10 commits into
bvaughn:masterfrom
fazouane-marouane:mar/scrollable

Conversation

@fazouane-marouane

@fazouane-marouane fazouane-marouane commented Apr 30, 2020 •

Copy link
Copy Markdown

Hi !

The issue of having custom scrollbars with react-virtualized is a pending issue #1275 .
It may be possible to do it today with Lists but it's kind of impossible/hacky to do it for Grid or MultiGrid (by subclassing Grid and using React.cloneElement for instance).
This PR solves this.

Screenshot 2020-04-30 at 17 19 51

checklist:

  • The existing test suites (npm test) all pass
  • For any new features or bug fixes, both positive and negative test cases have been added
  • For any new features, documentation has been added
  • For any documentation changes, the text has been proofread and is clear to both experienced users and beginners.
  • Format your code with prettier (yarn run prettier).
  • Run the Flow typechecks (yarn run typecheck).

@fazouane-marouane

Copy link
Copy Markdown
Author

Closing this as it hasn't been reviewed since its creation (9 months)

@bvaughn

bvaughn commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Sorry for the very long delay in getting back to you about this!

I've just finished a large update for version 10 of react-virtualized (#1900): the library has been rewritten in TypeScript, every component is now a function component (React 18+ is required), and the source moved from source/ to lib/. Because of that, this PR can't be merged as-is. Version 10's Grid and MultiGrid still don't support custom scrollbar components.

If you're still interested in this change, I'd welcome a new PR with it rebased onto the version-10 branch (or onto master once version 10 has been merged).

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