-
Notifications
You must be signed in to change notification settings - Fork 586
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: programmatically select active items #223
feat: programmatically select active items #223
Conversation
if possibly merged in, a thing that is left is to update |
e44b1f0
to
c6f6298
Compare
@Namek ok, looks really good to me!
|
Add `active` property which describes currently selected items. Remove `initData` property since everything works on top of `active`.
c6f6298
to
bb39a1b
Compare
@valorkin done, that should be it. To make it more angular-way we should enable two-way databinding by merging In case you didn't know what I'm talking about, these two snippets are equal:
(source: https://angular.io/docs/ts/latest/guide/forms.html) But I think if |
Any timeline when this will be merged and released to NPM? |
Usually Monday - Tuesday is release days On Fri, Jun 3, 2016, 12:51 Achim Hasenmueller [email protected]
|
When will this change merged? Seems it is still open |
Am I wrong? I don't see the changes published to NPM. |
Again, any timeline on when this will be merged in? |
Any update one when it will be published to npm? |
I'm releasing a product and really need this now. |
Hey guys, thanks a lot for this awesome directive! I could not make [active] property works. I have tried with everything Array, any[], cast element as SelectItem but nothing, there is no way for me to set the directive with initial data. It seems to be that this issue is related to that. We are using this in PRD so the sooner you could fix this or merge the fix, will be more than appreciated. Thanks in advance. |
Add
active
property which describes currently selected items.Remove
initData
property since everything works on top ofactive
.