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

Adicionar a opção para escolher a escala numérica a ser usada #24

Closed
wants to merge 3 commits into from

Conversation

guesant
Copy link
Contributor

@guesant guesant commented Sep 27, 2019

Implementação:

  • A função split agora aceita um novo parâmetro, scale. Se scale for igual a "long",
    o que antes era dividido em ['1', 000', '000', '000'] agora será ['1 000', '000', '000']

  • Se a função name receber um array em que algum item seja maior de que três algarismos (como no caso anterior), ele irá chamar recursivamente função gt1000, até que todos os parâmetros sejam menores que mil.

Acredito que resumidamente seja isso. Eu tentei manter ao máximo o estilo já presente no código. Caso veja algo diferente, saiba que foi o meu editor de texto ;)


Referente a issue #19

@theuves
Copy link
Member

theuves commented Sep 28, 2019

Muito bom!

@theuves
Copy link
Member

theuves commented Sep 28, 2019

Vou verificar com mais atenção assim que possível e fazer o merge.

@theuves
Copy link
Member

theuves commented Sep 28, 2019

Mas o que você acha de deixar a escala longa como padrão, com exceção ao Brasil (em { locale: 'br' }), que aparentemente é o único país que usa a escala curta?

@guesant
Copy link
Contributor Author

guesant commented Sep 28, 2019

  • long (valor padrão, com exeção em { locale: 'br' }) - ...
    o que você acha?

@guesant
Copy link
Contributor Author

guesant commented Sep 28, 2019

Feito!

@guesant
Copy link
Contributor Author

guesant commented Sep 30, 2019

Se for fazer o Merge em outubro, eu posso mandar o PR de novo 😅

@theuves
Copy link
Member

theuves commented Sep 30, 2019

Oi @guesant,

Sem pressa, assim que possível vou analisar tudo com mais cuidado.

@theuves
Copy link
Member

theuves commented Apr 27, 2020

Fiz um outra implementação (já "merged") onde as alterações são feitas diretamente na lista dos números.

@theuves
Copy link
Member

theuves commented Apr 27, 2020

Tenho que rever ainda a escala longa como padrão dos outros idiomas.

Na minha implementação a escala curta ficou como padrão de todos os idiomas.

@theuves theuves closed this Apr 27, 2020
@theuves
Copy link
Member

theuves commented Apr 27, 2020

Obrigado pelo Pull Request.

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

Successfully merging this pull request may close these issues.

2 participants