Bug/cleave input prefix & required - [merged]

[01oseluiz]

In GitLab by @fernando.jsts on Apr 6, 2022, 18:40

Merges bug/cleaveInput-prefix-required -> master

Comportamento anterior: havia um tratamento para retornar o valor do cleave input sem máscara, mas a leitura do required estava sendo em cima do valor ainda mascarado, então quando houvesse um prefixo ou sufixo, o campo não conta mais como vazio e ignora a validação de estar vazio.

O fix atual pretende invalidar também se o valor do input não estiver vazio (ou seja, tiver prefixo ou sufixo), mas o valor sem máscara estiver.

closes #50

[01oseluiz]

added 43 commits

• f266f9ef...47e89106 - 42 commits from branch master
• d5a4fa30 - Merge remote-tracking branch 'origin/master' into bug/cleaveInput-prefix-required

Compare with previous version

[01oseluiz]

Aqui vc acaba acoplando as duas coisas, o input não pode saber que o cleave existe. De novo problema de inteface que falamos. A ideia é que as atribuições sejam extremamente isoladas e consistentes. Imagine que você usa essa lib, mas não tem acesso a modificar ela, se vc quiser usar outro mask que não o cleave, como vai fazer?

[01oseluiz]

por que bind se esse atributo nem é alterado pelo "Input"?

[01oseluiz]

requested review from @01oseluiz

[01oseluiz]

In GitLab by @fernando.jsts on Apr 7, 2022, 19:56

Commented on src/components/formFields/Input/Input.svelte line 59

changed this line in version 3 of the diff

[01oseluiz]

In GitLab by @fernando.jsts on Apr 7, 2022, 19:57

Commented on src/components/formFields/CleaveInput/CleaveInput.svelte line 171

changed this line in version 3 of the diff

[01oseluiz]

In GitLab by @fernando.jsts on Apr 7, 2022, 19:57

added 4 commits

• b64e2ac7 - Cleave input (refactor): Undo unecessary changes
• 21f58604 - Input (refactor): simplify code logic
• 83077996 - CleaveInput (fix): Make cleave input validation work
• e6d2199f - Real example (docs): Alter CleaveInput masks example

Compare with previous version

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:41

Commented on src/components/formFields/CleaveInput/CleaveInput.svelte line 171

Abordagem foi completamente alterada, agora realmente o mexido no input foi apenas para melhorar o código mas a funcionalidade se concentra toda no cleave

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:41

Commented on src/components/formFields/Input/Input.svelte line 59

Abordagem foi completamente alterada, agora realmente o mexido no input foi apenas para melhorar o código mas a funcionalidade se concentra toda no cleave

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:41

resolved all threads

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:41

removed review request for @01oseluiz

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:42

added 11 commits

• e6d2199f...c3cf0931 - 10 commits from branch master
• ff080106 - Merge branch 'master' of...

Compare with previous version

[01oseluiz]

In GitLab by @fernando.jsts on Apr 8, 2022, 10:52

added 1 commit

• 36df624e - Input (fix): Changed function changes "isValid" property

Compare with previous version

[01oseluiz]

requested review from @01oseluiz

[01oseluiz]

added 3 commits

• cb819035 - Revert "Input (refactor): simplify code logic"
• e08ba6de - Revert "Input (fix): Changed function changes "isValid" property"
• 97d9d915 - input (refactor): change variable invalid to "visallyInvalid"

Compare with previous version

[01oseluiz]

mentioned in commit be7c1ffb2c51bb8cb1635769727313608eecf8e1