-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Update ja.js #2558
Update ja.js #2558
Conversation
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.
fixed lint error
lib/locales/ja.js
Outdated
'lat:': '緯度:', // traces/scattergeo/calc.js:48 | ||
'lon:': '経度:', // traces/scattergeo/calc.js:49 | ||
'q1:': 'q1:', // traces/box/calc.js:130 | ||
'q3:': 'q3:', // traces/box/calc.js:131 |
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.
Are q1 and q3 OK left in English? These are the first and third quartiles, in a box plot.
Thanks @jins-tkomoda - can you recommend another Japanese speaker to review? |
The two newest keys (which some of our first translations haven't updated to add) didn't make it into your list, could you add them? These are: |
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.
- added q1/q3 translations
- added missing 2 other translations
@yuji-uema @mbfts-ts could you check the translations? |
@jins-tkomoda |
Thanks @mbfts-ts for reviewing. And thanks @jins-tkomoda for the translations and edits! |
Add Japanese dictionary translations. (sorry, i couldn't update label to "type: translation",,, )