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

Virtualize dropdown options list #284

Merged
merged 21 commits into from
Aug 12, 2021

Conversation

dawsonbooth
Copy link
Contributor

Uses react-virtuoso to set up a windowed options list container that can handle items with variable, dynamic height. This is an apparently complicated problem that required a complicated solution, and custom logic to avoid this dependency could take a while to put together.

Closes #184

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 10, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 2c9c7a0:

Sandbox Source
React Configuration

@dawsonbooth dawsonbooth marked this pull request as ready for review August 10, 2021 17:13
Copy link
Contributor

@aVileBroker aVileBroker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only other thing is if we can simulate and test the virtualization (we should be able to use the same mock class/function that Harrison is using for the flippy floppy dropdown)

src/components/Dropdown/Dropdown.stories.tsx Outdated Show resolved Hide resolved
src/components/Dropdown/Dropdown.stories.tsx Show resolved Hide resolved
src/components/Dropdown/Dropdown.tsx Outdated Show resolved Hide resolved
src/components/Dropdown/Dropdown.tsx Outdated Show resolved Hide resolved
@dawsonbooth dawsonbooth merged commit f35fb1d into Headstorm:master Aug 12, 2021
@dawsonbooth dawsonbooth deleted the 184-virtual-dropdown branch August 12, 2021 21:02
@HeadstormOps
Copy link

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dropdown | Virtualize options list for better performance of long lists
3 participants