onchange event

[arindrajitb]

Hi there,
is there an onChange event available in the directive? I could not find one. If not do you have any plan to introduce one?
regards
Ari

[ruhley]

Currently if I need to do something on change I use $watch() (e.g. https://github.com/ruhley/angular-color-picker/blob/master/examples/app.js#L9-L11).

I could easily add in an onchange event though if you would prefer that.

[arindrajitb]

i'm using $watch at this point to get it going. But watch doesn't
distinguish between programmatic change and change by user. So i had to use
two sets of variable to identify whether change is coming from UI or from
back end. overall its pretty cumbersome as my page has two/three of this
directive.
It would be easier to use if the directive comes with a onchange similar
behavior like ng-change.
Also watch get fired multiple time when cursor is moved over the color
selection pallet. Again i have done some workaround( queuing to cancel lout
this events ). Ideally on change event should be fired when the color
selection is done. Dont know if that's expected from the directive or not.

regards
Ari

On 29 December 2015 at 21:53, Ruhley notifications@github.com wrote:

Currently if I need to do something on change I use $watch() (e.g.
https://github.com/ruhley/angular-color-picker/blob/master/examples/app.js#L9-L11).

I could easily add in an onchange event though if you would prefer that.

—
Reply to this email directly or view it on GitHub
#47 (comment)
.

[ruhley]

I have implemented a solution at commit 88a00de and release in v0.8.0.

Tell me if it works well or not. You can reopen if it doesn't suit

[arindrajitb]

Brilliant genius! It works like charm.
Really appreciate your quick update on this.

thanks and regards
Ari

On 29 December 2015 at 23:20, Ruhley notifications@github.com wrote:

Closed #47 #47.

—
Reply to this email directly or view it on GitHub
#47 (comment)
.