Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Limpar o campo #34

Open
johnvoloski opened this issue Jun 7, 2013 · 16 comments
Open

Limpar o campo #34

johnvoloski opened this issue Jun 7, 2013 · 16 comments

Comments

@johnvoloski
Copy link
Collaborator

Colocar uma opção de clear no options da máscara para caso não seja preenchido toda a máscara exemplo 'nnn.nnn.nnn-nn' (CPF) ao sair do campo ele seja limpado.

@fabiomcosta
Copy link
Owner

Eu vi essa feature em algumas libs quando implementei o meiomask e sempre achei isso ruim em termos de usabilidade.
As vezes usuario esquece um numero e vai para o proximo e acaba perdendo todo o conteudo do input.
O que voce acha?

@johnvoloski
Copy link
Collaborator Author

Bom dia Fábio,

Entendo, acredito que por este lado tu esteja correto, mas também vai da visão de quem "compra" o sistema, nesse projeto o PO acredita que é melhor limpar os campos caso estejam incorretos pro usuário ver que está com problema, pois é um campo que contém um serial e realmente fica mais fácil quando incorreto apagar automáticamente. Acredito que essa funcionalidade serviria para alguns tipos de campos. Por isso achei que podia ser algo customizável, o default ser não limpar, mas ter a opção.

@jcspader
Copy link

Estou com a mesma situação do @johnvoloski
Acho que poderia criar uma opção configurável e deixar este opcional.

Quando true, no evento "blur", ele apagaria o valor do campo se não fosse valido.
@fabiomcosta, já tem algum método que valide se o valor esta valido com a mascara?

@fabiomcosta
Copy link
Owner

ping @johnvoloski

@johnvoloski
Copy link
Collaborator Author

@jcspader , eu fiz essa implementação no plugin, de conseguir setar um evento próprio "blur", em seguida vou enviar o pull request dessa implementação.

Concordo com o Fábio acima, em não limpar automáticamente pois se o usuário esquece teria de redigitar, porém o evento resolveria, se for o caso no blur serviria para implementar o desenvolvedor implementar isso.

@jcspader
Copy link

@johnvoloski ai é bonito hem! Agradeço a vocês o esforço.
O plugin é ótimo!

@jcspader
Copy link

@johnvoloski Não localizei o pull, você o fez?

@johnvoloski
Copy link
Collaborator Author

@jcspader , Acabei de fazer o pull request :D

@jcspader
Copy link

@johnvoloski Eu vi pela web que você criou suporte aos eventos Blur e Focus.
Mas não achei como ele vai fazer para que quando ele fizer o Blur e o campo não estiver condizente com a mascara, ele apague o valor.

johnvoloski added a commit that referenced this issue Jun 25, 2013
Issue #34 - Implements onFocus and onBlur Callbacks
@jcspader
Copy link

@johnvoloski , @fabiomcosta Tem algum método que retorne se o valor é a valido para a mascara?

@jcspader
Copy link

Realizei a implementação no meu projeto.
Mas acredito que que a minha sugestão para implementar seria bastante útil.

EventBlurClearMask = function (elem, ev) {
    if ($(elem).data('mask').mask.length != $(elem).val().length)
        $(elem).val("");
};

   $('input').setMask({
        mask: "999.999.999-99",
        onBlur: that.EventBlurClearMask
    });

@johnvoloski
Copy link
Collaborator Author

@jcspader , estava realizando uma mudança na estrutura do plugin, agora ele tem testes automatizados e integração continua, a seguir vou começar a resolver as issues pendentes, com prioridade em bugs, e para funcionar perfeitamente em mobile, tables, etc... estou resolvendo com o @fabiomcosta se vamos implementar no plugin está funcionalidade, mas acredito que em breve temos uma posição, agradeço tua ajuda por inclusive nos postar uma idéia de como implementar.

Att.

@johnvoloski
Copy link
Collaborator Author

@jcspader, está implementação irá entrar na próxima versão do plugin 1.1.12.

Att.

@jcspader
Copy link

jcspader commented Feb 5, 2014

@johnvoloski você já liberou esta alteração?

Abraço.

@johnvoloski
Copy link
Collaborator Author

@jcspader

Na inicialização da mascara é possível setar um handler no foco ou saida, pra implementar esse "clear".

https://github.com/fabiomcosta/jquery-meiomask/blob/master/src/meiomask.js#L146
https://github.com/fabiomcosta/jquery-meiomask/blob/master/src/meiomask.js#L147

@jcspader
Copy link

jcspader commented Feb 5, 2014

@johnvoloski Isso eu sei.. até exemplifiquei no post. Mas a sugestão é deixar isso nativo na biblioteca como uma configuração/propriedade de true/false, para fazer na forma automática.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants