Allow specifying overscanThreshold as prop

Open
#3,929 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
55/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
react, typescript
Domain
frontend

Research direction

Start by locating the DataGrid virtualization configuration and the existing overscanThreshold default, then trace how DataGrid props reach virtualization. Expose the value per grid while preserving the default of 4, and verify that fast scrolling renders the configured number of rows ahead.

Written by the indexing model from the issue text.

Description

Use case

My grids render rows that fetch their own data via RTK Query on mount. This works well with enableVirtualization, but the default overscanThreshold of 4 is a little too low in practice. When users scroll quickly, many unloaded rows come into view and they briefly display loading placeholders. I can afford to render more rows ahead of the viewport, which would trigger earlier fetches and eliminate most of this flicker.

Proposed solution

Expose overscanThreshold as a DataGrid prop so it can be configured per grid. This would let developers balance performance and perceived smoothness based on their actual data-loading behaviour.

Dominant language
TypeScript
Stars
7.7k
Forks
2.2k
Avg merge
1d 10h
Merged PRs (30d)
30

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Comcast/react-data-grid

All issues in Comcast/react-data-grid

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.