amarkes / br-masker-ionic-3

mascara para inputs
MIT License
110 stars 32 forks source link

Campos estão aceitando letras #30

Closed wesleycavalcante closed 6 years ago

wesleycavalcante commented 6 years ago

Alterei o arquivo brmasker-ionic-3.js linha 124 para:

var exp = /-|.|\/|(|)|\,|*|+|\@|#|\$|\&|\%|\:| |[A-Za-z]/g;

amarkes commented 6 years ago

O que de fato esta querendo fazer? Seja mais especifico, talvez uma solução para ti, pode não ser uma solução para os outros.

wesleycavalcante commented 6 years ago

Minha máscara era 00.0000.00000000-0 e estava permitindo colocar letras quando eu queria apenas números

amarkes commented 6 years ago

Existem pessoas que usam a mascara com letra, fazendo dessa forma, prejudica os que já usam, o que pode ser feito é passar mais uma configuração informando se usaria so numero ou so letra ou os dois...

verissimomanoel commented 6 years ago

Eu acredito que seria legal colocar esse parametro mesmo, pois por exemplo: máscara de CPF só aceita número, não faz sentido aceitar letras. Eu estou tentando usar a biblioteca e estou tendo a mesma dificuldade.

verissimomanoel commented 6 years ago

Uma maneira de resolver o problema é fazer com no celular só apareça o teclado numérico, assim o usuário não conseguirá digitar letras. Eu fiz isso aqui e resolveu o meu problema, segue abaixo o exemplo: <ion-input type="text" pattern="\d*" name="login" title="Login" [brmasker]="{mask:'000.000.000-00', len:14}">

jeffersonmello commented 6 years ago

@verissimomanoel poderia explicar melhor como utilizar os teclados numéricos/textuais?

amarkes commented 6 years ago

@verissimomanoel para cif já existe um tratamento para isso [brmasker]="{person: true}"

verissimomanoel commented 6 years ago

Bom o problema relatado era que ao colocar uma máscara, por exemplo de CPF, aceitava digitar letras. Então se colocar o parametro pattern="\d*" o ionic abre somente o teclado numérico e nesse caso além de resolver o problema melhora a usabilidade do app.

amarkes commented 6 years ago

Fiz uma nova versao com o campo type | type | string | Optional default 'all' |

alfa => alfanuméricos num => números all => aceita a porra toda

Exemple

[brmasker]="{mask:'00/00/0000', len:10, type:'num'}"
verissimomanoel commented 6 years ago

@amarkes ainda com esse parâmetro, recomendo o uso do pattern, pois como só vai aceitar número no campo então, abrir somente o teclado numérico ajuda na usabilidade.

amarkes commented 6 years ago

Caso seja resolvido, podem fechar a issue?

verissimomanoel commented 6 years ago

Por mim já pode fechar.

jeffersonmello commented 6 years ago

@verissimomanoel obrigado pela explicação, por min closed

wesleycavalcante commented 6 years ago

Por mim pode fechar.

juniorsistemasufms commented 5 years ago

Pessoal acompanhei as postagem porém estou tentando fazer o type num do CPF no formato 999.999.999-99 porém abrindo somente o teclado numérico.. e não consegui. se coloco ion-input para number abre o teclado numerico porém a mascara não funciona.(obviamente acho que ela faz o que deve não permitir caracteres)

contudo queria ver um meio termo

mascara :: 999.999.999-99 abrindo o teclado numerico somente será q tem como ?

amarkes commented 5 years ago

Junior tenta o type tel