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

[WIP] feat: autocomplete limit #1163

Closed
wants to merge 3 commits into from
Closed

Conversation

tujoworker
Copy link
Member

Closes #1100

@tujoworker tujoworker requested a review from vron December 3, 2021 13:35
@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 3, 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 436a08f:

Sandbox Source
eufemia-starter Configuration

@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 3, 2021

✅ DNB Eufemia Portal deploy preview ready

Copy link
Contributor

@vron vron left a comment

Choose a reason for hiding this comment

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

lgtm - but not very knowledgeable about the general codebase..

@tujoworker
Copy link
Member Author

Right now I play arround of not using a limit – but rather a driver for a thid party virtual scroller – because it works ok when filtering, because of the re-ordering – but what of you show everthing at first? A list with 10 000+ will then be show to scroll through.

Now, with a virtualized scroller, only the needed items are rendered – its just, have a almost Plug & Play method would be best – any thoughts on that?

@tujoworker tujoworker changed the title feat: autocomplete limit [WIP] feat: autocomplete limit Dec 16, 2021
@tujoworker tujoworker marked this pull request as draft December 16, 2021 21:38
@tujoworker tujoworker force-pushed the feat/autocomplete-limit branch from 0cb10a6 to 436a08f Compare October 27, 2022 18:27
@tujoworker tujoworker closed this Dec 5, 2022
@tujoworker tujoworker deleted the feat/autocomplete-limit branch March 7, 2023 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Autocomplete limit rendered items
2 participants