fix(geo): Layer color picker do not change the fill/stroke color #1509
Conversation
| * The emitted value represent the current chosen color in the picker | ||
| * The value is emitted by default in RGBA, or based on the input 'outputFormat' value. | ||
| */ | ||
| @Output() colorChange: EventEmitter<string> = new EventEmitter<string>(); |
There was a problem hiding this comment.
Cette composante est une variant d'un FormField elle fonctionne avec le ReactiveForm d'Angular. Lorsqu'on lui passe un FormControl, on peut s'abonner au changement au niveau du FormControl. Voir l'exemple dans le prochain commentaire
There was a problem hiding this comment.
Oui sauf que dans ce cas la, pour setter le form, il faut cliquer sur ok ou appliquer, donc le changement n'est pas dynamique.
| @@ -14,6 +14,7 @@ <h2 mat-dialog-title class="mat-typography"> | |||
| </span> | |||
| <igo-color-picker-form-field | |||
| formControlName="fill" | |||
| (colorChange)="setLayerFillColor($event)" | |||
There was a problem hiding this comment.
On pourrait avoir les changements de valeurs ainsi dans le Typescript de la composante:
| (colorChange)="setLayerFillColor($event)" | |
| this.form | |
| .get('fill') | |
| .valueChanges.subscribe((color) => this.handleFillColor(color)); |
There was a problem hiding this comment.
Il faut cliquer sur Appliquer/OK pour récupérer la couleur
There was a problem hiding this comment.
Vraiment plus convivial pour choisir une couleur, surtout si des superpositions/conflits visuel de couleur sont possible.
Please check if the PR fulfills these requirements
What is the current behavior? (You can also link to an open issue here)
related to issue #1508
What is the new behavior?
Fix it?
Does this PR introduce a breaking change? (check one with "x")
If this PR contains a breaking change, please describe the impact and migration path for existing applications:
Other information: