-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Use SelectableList in RelationPicker, SingleEntitySelectBase and MultipleEntitySelect #2949
Use SelectableList in RelationPicker, SingleEntitySelectBase and MultipleEntitySelect #2949
Conversation
@Kanav-Arora we were having linter issues (it was broken). I've used your PR to fix it :p Could you pull your branch (I've pushed on it), run "yarn" and then "yarn lint" (from front folder or yarn nx lint twenty-front from root) |
Regarding your fix and the refactoring, LGTM, I will give another try once linter is fixed! |
Hi @charlesBochet |
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.
Thank you! @Kanav-Arora, very helpful as usual!
Closes #2747
SelectedItemId state is to be updated if and only if there's a possibility of new id. So now up arrow won't work if 0th item is selected. Same for all other keys.