search

Brasil-Nihon-Kiin / nihonkiin.com.br

A infraestrutura da casa do Go brasileiro, a Brasil Nihon Kiin.
http://nihonkiin.com.br/
Other
8 stars 3 forks source link

Altera tipo campo nascimento para input date #24

Closed laercioskt closed 3 years ago

laercioskt commented 3 years ago

resolve #20

Pesquisei sobre as limitações em relação ao uso do input date (https://developer.mozilla.org/pt-BR/docs/Web/HTML/Element/input/date) e realmente parece ser bem chato lidar com isso.

De qualquer forma, implementei e testei em alguns browsers (Chrome Linux, Firefox Linux e Chrome no Android) e me pareceu ok.

A limitação é que a formatação da data deve respeitar a localização definida no navegador do usuário, então, não é possível deixar sempre no formado dd/mm/yyyy.

Gostaria que avaliassem e até testassem em outros browsers para ver se é ok integrar esse desenvolvimento na main.

psygo commented 3 years ago

O que você quer dizer exatamente com "testei"? O formulário e a lógica em TS estão funcionando corretamente? O email está sendo transcrito corretamente?

laercioskt commented 3 years ago

Que as validações de preenchimento da data se mantinham, que o preenchimento não fosse prejudicado no celular ou desktop e que o email foi transcrito corretamente.

Porém, acredito que faltou informação, eu não quis mexer no formato padrão de saída do campo desse tipo, ou seja, ficou "yyyy-mm-dd", mas pensando bem acho que seria interessante formatar mesmo.

Vou melhorar assim que tiver um tempinho.

laercioskt commented 3 years ago

@psygo Repare que fiz alguns ajustes conforme comentei, mas acabei me confundindo aqui e enviei direto para a main. Dado o uso do site e as alterações feitas não vi tanto problema em deixar, e não reverti para que a revisão seja feita antes. Caso não concorde com alguma alteração posso reverter e a gente volta a discutir aqui, ok?

laercioskt commented 3 years ago

Commits: https://github.com/Brasil-Nihon-Kiin/nihonkiin.com.br/commit/4b32c4749aaf59743b57ffc293d5ec32d9c2591e https://github.com/Brasil-Nihon-Kiin/nihonkiin.com.br/commit/5136918f00e4157d478d4134673cd0a7c1f723aa https://github.com/Brasil-Nihon-Kiin/nihonkiin.com.br/commit/e7e5e69c2f7193e53441f565c65e070921679454 https://github.com/Brasil-Nihon-Kiin/nihonkiin.com.br/commit/6dbe8f02218385d2d9c8c96df6f9c5f12e06e0ac

psygo commented 3 years ago

Boa, achei bacana que, utilizando um date legítimo, no celular, ele abre algo interativo para se escolher a data inclusive. Mas, no browser, ainda está aparecendo para mim como mm/dd/yyyy, você sabe como substituir o placeholder e a formatação?

Deixei alguns comentários nos commits com sugestões.

Aproveita para deletar o arquivo de teste de exemplo que eu tinha deixado lá, agora que a gente tem um exemplo legítimo.

laercioskt commented 3 years ago

ainda está aparecendo para mim como mm/dd/yyyy, você sabe como substituir o placeholder e a formatação?

Usando html puro não é possível fazer isso, o input>>date vai se basear na configuração do browser, então, se estiver em português do brasil, vai aparecer dd/mm/aaaa. O retorno do campo é sempre yyyy-mm-dd independentemente da configuração do browser.

Também não sei se é possível usando algum framework e, por enquanto, acho que a gente não precisa se preocupar com isso.

Ah, já fiz as alterações sugeridas nos outros commits no commit https://github.com/Brasil-Nihon-Kiin/nihonkiin.com.br/commit/3bc1174cf4dd540972ad7dd401a4d1518a9d8d21