Proposal: remove this.colorPickerChange from the directive in ngOnInit()

[alberplz]

Proposal: remove this.colorPickerChange from the directive in ngOnInit()

[no-more]

Hi,

I think I have the same issue, change event is triggered when my components init, and so I mark my form as dirty by error. It would be great to not trigger the event on input changes or at least not on creation.

Thanks

[maartentibau]

@Alberplz a new option isn't needed. I did this and that works perfectly!

ngOnInit() {
  var hsva = this.service.stringToHsva(this.colorPicker);
  if (hsva == null) {
    hsva = this.service.stringToHsva(this.cpFallbackColor);
  }

  if (this.created) {
    this.colorPickerChange.emit(this.service.outputFormat(hsva, this.cpOutputFormat));
  }
}

FYI: it's better to use let than var in typescript.

[wartab]

@renaudaste angular2-color-picker is not maintained anymore. Use this instead: https://github.com/zefoy/ngx-color-picker

It is a drop in replacement for this package and should work out of the box with bug fixes and added features. Should your use case not be covered yet in it, feel free to report it there.

[Alberplz]

If the event is only fired when the color input format doesn't match with the color output format?

I can change that line by:

let output = this.service.outputFormat(hsva, this.cpOutputFormat);
if (output !== this.colorPicker) {
this.colorPickerChange.emit(this.service.outputFormat(hsva, this.cpOutputFormat));
}

[maartentibau]

@Alberplz that's a solution for components that are dynamically created, but if you use the renderer to alter some stuff on dynamic created components, then it will still use the values created by color picker instead of the pre-set colors.

[Alberplz]

Then a new option to control that line could be a good solution

[Alberplz]

I have not tested with components dynamically created but it's nice!! :)

Yes....yesterday I committed that var

[BrainCrumbz]

any update on this? Thank you

[renaudaste]

Any update on this feature ? (would be great not to have to fork).
Current code on master is still : https://github.com/Alberplz/angular2-color-picker/blob/master/src/color-picker.directive.ts#L64-L71
Thank you for your great work 👍