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

Allow deselection #59

Open
ebidel opened this issue Apr 13, 2016 · 2 comments
Open

Allow deselection #59

ebidel opened this issue Apr 13, 2016 · 2 comments

Comments

@ebidel
Copy link

ebidel commented Apr 13, 2016

I find myself increasingly wanting a way to deselect an item once it is selected. Currently, the only want to do this is use mult, but allowing multiple selection is not always desired.

In the past, I've worked around this by using the iron-deselected event manually deselected the item in js :(

@bicknellr
Copy link
Contributor

bicknellr commented Apr 18, 2016

selector.select(undefined) should do it. There's kind of a mismatch between IronSelectableBehavior and IronMultiSelectable behavior though: .select(undefined) for a multi-selectable will actually push undefined to selectedValues. :/

@bicknellr
Copy link
Contributor

bicknellr commented Apr 19, 2016

Calling this p2 just because there's a pretty straight-forward workaround but this is staying open / should be kept in mind when we get to the point that we can make API changes. The current state of iron-selector is pretty poorly defined and crappy to work with concerning this case. I think IronSelectableBehavior and IronMultiSelectableBehavior might need to be split a bit more than they are now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants