md2 icon indicating copy to clipboard operation
md2 copied to clipboard

Colorpicker does not triggered change event with a wrong or empty value

Open dimzeta opened this issue 7 years ago • 2 comments

Bug, feature request, or proposal:

Bug

What is the expected behavior?

When we put a wrong or an empty value, the model should be have this new value.

What is the current behavior?

The model has a new value only when this is a correct value.

What are the steps to reproduce?

My plunker : http://plnkr.co/edit/481ewmoT6twi0sYulBWS?p=preview

Set a new color to see the value change in the model, then remove it. The old value still here. Now put a wrong value, like "WrongValue", nothing happens. This is a little annoying with an ngForm behind.

Providing a Plunker (or similar) is the best way to get the team to see your issue. Plunker template: http://plnkr.co/edit/rQmUz8WYRh5Vz4gwY8E4?p=info

What is the use-case or motivation for changing an existing behavior?

Which versions of Angular, MD2, OS, browsers are affected?

Is there anything else we should know?

dimzeta avatar Jul 05 '17 00:07 dimzeta

@dimitribocquet If you set wrong value/blank value then it will set #000000 as a default color.Wrong value can not bind in the model, it will take only valid color value.

krupa310 avatar Jul 10 '17 11:07 krupa310

That sounds a logic behavior, but when I use a ngForm with it, the form is never warned that the value is wrong, because as you said it hasn't changed. So from UX view, we can still submit the form (form.valid == true) with a wrong value.

Maybe we should be able to allow to bind it in the model even if it's wrong, with an @Input parameter?

dimzeta avatar Jul 10 '17 13:07 dimzeta