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

feat(ffe-searchable-dropdown-react): Add support for tens of thousands of options #1106

Merged
merged 1 commit into from
May 21, 2021

Conversation

torkjels
Copy link
Contributor

Using react-virtualzed we only render the list options that are dislayed to the user.
This enables us to go from at most a few thousand options to virtually an unlimited amount.
This implementation has been tested with 300000 options on Chrome.

This aims to solve the same issue that the highCapacity prop of AccountSelector was
introduced to solve in 93f2189


Jeg har testet dette på TalkBack og sammenlignet med prod, og det ser ut til å funke likt.

@coveralls
Copy link

coveralls commented May 12, 2021

Coverage Status

Coverage increased (+0.07%) to 79.574% when pulling d5dc190 on virtualize-searchable-dropdown into 3d23895 on develop.

@pethel
Copy link
Contributor

pethel commented May 13, 2021

Jag startade op dette lokalt og opplever hvis jag navigerar nedøver med pilen på tastaturet så funker det ikke som forventa. Tror jag iaf. Hvis jag er på "Navn 5" og trykker på pil ned så havner jag på "Navn 3" men fler alternativer blir synliga. Oppplever kun problemet i listan med veldigt mange alternativer.

Den ser ut att hanterna olika højder på custom elementer fint

@MariaOverlierBerg MariaOverlierBerg self-assigned this May 18, 2021
@MariaOverlierBerg
Copy link
Contributor

Jag startade op dette lokalt og opplever hvis jag navigerar nedøver med pilen på tastaturet så funker det ikke som forventa. Tror jag iaf. Hvis jag er på "Navn 5" og trykker på pil ned så havner jag på "Navn 3" men fler alternativer blir synliga. Oppplever kun problemet i listan med veldigt mange alternativer.

Den ser ut att hanterna olika højder på custom elementer fint

Dersom pilen til musa hovret over et av elementene i lista, så hoppet av og til fokus dit, så fjernet funksjonaliteten at man setter index ved hovring. Men det sitter fortsatt igjen litt hopping hvis man kjapt piler seg ned forbi 7. element i lista, men den ender opp på riktig element sånn vi ser det. Det er jo litt uheldig, men tror ikke det er så mange som piler så fort at det blir et problem.

…s of options

Using react-virtualzed we only render the list options that are dislayed to the user.
This enables us to go from at most a few thousand options to virtually an unlimited amount.
This implementation has been tested with 300000 options on Chrome.

This aims to solve the same issue that the highCapacity prop of AccountSelector was
introduced to solve in 93f2189
Copy link
Contributor

@pethel pethel left a comment

Choose a reason for hiding this comment

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

Sjekket ut og testet på nytt. Ser ut og funke

@torkjels torkjels merged commit 5c22856 into develop May 21, 2021
@torkjels torkjels deleted the virtualize-searchable-dropdown branch May 21, 2021 11:33
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

5 participants