-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: add support for Devanagari & Bengali decimal numbering systems #6715
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
Conversation
Currently this is being tested by Express g18n team. Will come back with updating all the necessary documentation when it is validated. |
8cb1a5d
to
3013156
Compare
a79adcf
to
3013156
Compare
I am marking this as ready for review as it was finally tested by Express g18n team and confirmed that number field works well for Hindi and Bengali keyboards. @majornista I think this PR is affecting your PR: #6520, would you like me to add additional checks to mine? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks pretty good, thanks for the tests.
Co-authored-by: Robert Snow <[email protected]>
I no longer have the option to add additional reviewers, @snowystinger would you help me ping additional RS members for code review? |
Someone else on the team will review it, thank you for your patience |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, verified I was able to type various numeric values via IDE into the NumberField when Devanagari and Bengali was enabled
Closes #6623
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project:
Adobe Express