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

Virtualization for FluentAutocomplete #1647

Merged
merged 3 commits into from
Mar 7, 2024

Conversation

martinbu
Copy link
Contributor

@martinbu martinbu commented Mar 6, 2024

This PR implements the virtualization feature for FluentAutocomplete discussed here #1638

For me the implementation works fine and greatly improves the initial rendering performance and the reactivity when a search is done on that list. The improvement can be checked by changing the Virtualize Parameter in the Autocomplete with Many Items Demo example.

@vnbaaij vnbaaij enabled auto-merge (squash) March 7, 2024 13:30
martinbu added 3 commits March 7, 2024 14:58

Unverified

No user is associated with the committer email.
The MaximumOptionSearch can now be disabled by
providing a negative number.
Using virtualization for Autocomplete greatly improves the
initial rendering performance and rerendering performance
if the search text is changed if many items/options are present.
auto-merge was automatically disabled March 7, 2024 14:11

Head branch was pushed to by a user without write access

@martinbu martinbu force-pushed the feature/virtualize-autocomplete branch from 998e053 to 3c82f3b Compare March 7, 2024 14:11
@martinbu martinbu requested review from dvoituron and vnbaaij March 7, 2024 14:13
@vnbaaij vnbaaij enabled auto-merge (squash) March 7, 2024 14:15
@vnbaaij vnbaaij removed the request for review from dvoituron March 7, 2024 14:18
@vnbaaij vnbaaij dismissed dvoituron’s stale review March 7, 2024 14:19

I decided to not always use Virtualization

@vnbaaij vnbaaij merged commit 93576b3 into microsoft:dev Mar 7, 2024
2 checks passed
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.

None yet

3 participants