Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Implements new lookup for multiple selection #1231

Merged
merged 24 commits into from
Oct 4, 2017
Merged

Implements new lookup for multiple selection #1231

merged 24 commits into from
Oct 4, 2017

Conversation

pablosichert
Copy link
Contributor

#1138

See in action:

labels

pablosichert and others added 24 commits October 3, 2017 12:04
@pablosichert
Restructure getClassnames for clarity
996a02c
@pablosichert
Refactor dropdownRequest argument list to options object
6ef206d
@pablosichert
Sketch out <Labels> component
dee36d1
@pablosichert
Improve getClassNames signature for readability
124fb57
@pablosichert
Sketch out single labels
2ac820d
@pablosichert
Improve autocompleteRequest signature for readability
7bfd32b
@pablosichert
Add list with type-ahead suggestions
5060812
@pablosichert
Refactor patchRequest argument list to options object
b4cb8ec
@pablosichert
Implement removing label by pressing backspace
24c7874
@pablosichert
Only suggest labels that are not active yet
cac492c
@pablosichert
Use props.onChange instead of calling patchRequest directly
5c47d30
@pablosichert
Implement adding suggested label by clicking on it
5ecc937
@pablosichert
Only show suggestion list when widget is focused
d8a2639
@pablosichert
Add tabIndex to <Labels>
24e23e9
@pablosichert
Encapsulate <Suggestion> in own component to keep same onClick functi…
3af8a6d 
…on reference
@pablosichert
Resolve click on suggestion not registering before <Label> loses focus
765f40c
@pablosichert
Hook suggestions on key up so that it catches the last key input
d9a45fb
@pablosichert
Polish styling of <Labels>
25ce319
@pablosichert
Guard contenteditable from breaking by white-listing keys
b0829bd
@pablosichert
Submit first suggestion shown when hitting enter
31524ba
@pablosichert
Add handle to remove labels
597fe84
@pablosichert
Fix error when hitting enter and no label matches text input
e68294d
@pablosichert
Fix focus issues when selecting from dropdown
49da97e
@teosarca
Merge branch 'master' into dev-1138
4798518
@teosarca teosarca merged commit 75f3bf2 into master Oct 4, 2017
@teosarca teosarca deleted the dev-1138 branch October 4, 2017 22:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pablosichert @teosarca