concretesolutions / canarinho

Utilitários Android para padrões Brasileiros
Apache License 2.0
178 stars 42 forks source link

Correção do input com máscara de valor monetário R$ #17

Closed luisfernandezbr closed 6 years ago

luisfernandezbr commented 7 years ago

Fiz alguns testes para usar o input com a máscara de "R$" e encontrei o seguinte problema. Se o inputType fosse "number" o Editable removia o "R$" durante a digitação.

cs-victor-nascimento commented 7 years ago

Fala @luisfernandezbr! Muito obrigado pelo PR!

O CI quebrou por causa da configuração do Travis. Eu fiz uma atualização e subi para a master. Se você puder fazer um rebase na master atual pra deixar o CI rodar novamente aceitaremos o PR!

Se possível adicionar um teste para cobrir o código que incluiu seria bacana também!

Desculpe a demora em responder.

cs-victor-nascimento commented 7 years ago

@luisfernandezbr se estiver sem tempo e/ou se quiser eu posso commitar no seu fork para resolver os conflitos.

Cheers,

cs-victor-nascimento commented 6 years ago

ping :)

cs-victor-nascimento commented 6 years ago

Vou aceitar como está e resolvo os conflitos localmente antes de fazer um release!

Muito obrigado pela contribuição @luisfernandezbr!