-
Notifications
You must be signed in to change notification settings - Fork 87
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
Conversation
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
15ce3ce
to
d5dc190
Compare
There was a problem hiding this 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
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.