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

2 commits improving picker #117

Merged
merged 6 commits into from
Aug 29, 2020
Merged

Conversation

mkalantar
Copy link

hi
I updated .gitignore file to ignore vscode settings.
In the last commit a method is added to picker component that convert numbers to Farsi by checking the locale. This method removes font dependency for Farsi numbers.

@mkalantar mkalantar changed the title 2 commit improving picker 2 commits improving picker Aug 10, 2020
@talkhabi
Copy link
Owner

Hi @mkalantar
Thank you for collaborating on this project.

Can you please add a boolean prop to picker named convertNumbers ? ( which is false by default)
Then check the value of convertNumbers before replacing the numbers in fa.

@mkalantar
Copy link
Author

It's done.

Copy link
Owner

@talkhabi talkhabi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please refactor the mentioned lines.

@talkhabi talkhabi changed the base branch from master to feature/fa-numbers August 29, 2020 06:27
@talkhabi talkhabi merged commit 77aef14 into talkhabi:feature/fa-numbers Aug 29, 2020
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