search

EvanOxfeld / angular-selectize.js

Angular Selectize directive for a hybrid textbox + <select> box. Supports ngOptions and two-way bindings.
MIT License
34 stars 16 forks source link

Eval options #21

Closed jpmitche11 closed 10 years ago

jpmitche11 commented 10 years ago

Fixes problems when the ng-options has nested properties.

Here is a plunk demonstrating the issue: http://plnkr.co/edit/K4Lblj77W8NBMmthd2pP?p=preview

EvanOxfeld commented 10 years ago

Thanks @jpmitche11 -- nice fix!

Before we publish a new version of the directive, should the other places that $eval options use the function generated by $parse instead? I'll also add a test or two.

EvanOxfeld commented 10 years ago

Thanks again for the fix @jpmitche11 :zap:

I merged this pull request via 3815d6e55499d168d377849fa5b2f70a15594791 and I added some tests in 18d842dc53af24d44ee5ae7211a929b39b182fdc. Published to bower as 1.4.1.