po-angular icon indicating copy to clipboard operation
po-angular copied to clipboard

feat(datepicker): ativa mudança de estado também no evento `dateSelected`

Open wsteixeira opened this issue 3 years ago • 1 comments

A mudança de estado já ocorre no evento eventOnBlur e foi incluído também no evento dateSelected devido a ordem dos disparos dos eventos no programa po-dynamic-form-fields.component.ts:

  1. Quando a data é informada manualmente e depois teclado "TAB" o evento eventOnBlur ocorre antes do evento onChangeField.
  2. Quando a data é escolhida através do ícone/calendário e depois teclado "TAB", o evento eventOnBlur ocorre depois do evento onChangeField.

Na segunda situação o evento validate do componente page-dynamic-edit não é disparado, porque a mudança de estado só ocorrerá no evento eventOnBlur.

Fixes #1207

Datepicker

1207


PR Checklist [Revisor]

  • [ ] Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • [x] Código (Boas práticas, nome de variavéis/métodos, etc.)
  • [x] Testes unitários (Cobre a situação implementada e coverage está mantido)
  • [ ] Documentação (Clara, objetiva e com exemplos caso necessário)
  • [x] Samples (A implementação possui exemplo no Labs/Caso de uso)
  • [x] Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual? O evento validate não é disparado no componente page-dynamic-edit quando o datepicker é o primeiro componente a ser tocado.

Qual o novo comportamento? A mudança de estado também no evento dateSelected no componente, permite que o evento validate seja disparado no componente page-dynamic-edit quando o datepicker é o primeiro componente a ser tocado.

Simulação Usar o App para realizar a simulação.

wsteixeira avatar Feb 21 '22 22:02 wsteixeira

Boa tarde. Favor alterar commit de acordo com o nosso padrão. Sugestão:

feat(datepicker): emite onTouchedModel quando data é selecionada

Emite onTouchedModel quando data é selecionada pela primeira vez.

Fixes #1207

jcorrea97 avatar Feb 25 '22 17:02 jcorrea97