-
Notifications
You must be signed in to change notification settings - Fork 131
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
Select items #457
base: master
Are you sure you want to change the base?
Select items #457
Conversation
Based on the response from the Polymer team in Polymer/polymer#4699 (comment) would a PR that included a custom I guess one alternate possibility would be to have a stand alone component/repo that handled these responsibilities and then require it in |
Would that still block this PR on 1.x not having this capability? Can we move to a non-hybrid |
I would lean towards just baking |
Fixes #436
Relates to #361 (would make any batch selecting therein also not be slow)
Relies on:
Polymer/polymer#4699
Polymer/polymer#4705
Adds code paths and tests to support selecting all of the items in the list or batch selecting a subset of the list. Further the ideas introduces in #437, this relies on extensions to both Polymer 1.x and 2.x to empower
array-selector
with selectItems and selectAll functionality and forks the application of that code as needed in this hybrid component.