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

Initial setup for adding Hindi to the editor. #1703

Merged
merged 15 commits into from Jan 29, 2021

Conversation

abhieshekumar
Copy link
Contributor

Initial setup for adding Hindi to the editor #1701

I have verified that this pull request:

  • [✓] has no linting errors (npm run lint)
  • [✓] is from a uniquely-named feature branch and has been rebased on top of the latest develop branch. (If I was asked to make more changes, I have made sure to rebase onto develop then too)
  • [✓] is descriptively named and links to an ongoing issue #1701

@abhieshekumar abhieshekumar changed the title Initial setup for adding Hindi to the editor #1701. Initial setup for adding Hindi to the editor. Dec 5, 2020
@catarak
Copy link
Member

catarak commented Dec 7, 2020

I'd prefer to merge this in with the Hindi translation done, rather than just the setup completed.

@abhieshekumar
Copy link
Contributor Author

Sure thing. Working on that.

translations/locales/hi/translations.json Outdated Show resolved Hide resolved
translations/locales/hi/translations.json Outdated Show resolved Hide resolved
translations/locales/hi/translations.json Show resolved Hide resolved
@catarak
Copy link
Member

catarak commented Jan 5, 2021

@rt1301 I merged your changes to here! I'm going to close your other PR.

@catarak catarak mentioned this pull request Jan 5, 2021
3 tasks
@abhieshekumar
Copy link
Contributor Author

@catarak I have fixed the requested changes. Take a look.

@catarak
Copy link
Member

catarak commented Jan 6, 2021

Thank you @abhieshekumar! Everything looks good code-wise, so I'm going to solicit some feedback on this translation as you have requested 😄

"Title": "मदद",
"KeyboardShortcuts": "कीबोर्ड शॉर्टकट",
"Reference": "रिफरेन्स",
"About": "के बारे में"

Choose a reason for hiding this comment

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

"About": "अधिक जान्कारी"

Choose a reason for hiding this comment

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

के बारे में looks incomplete

"SignUp": "साइन अप",
"Auth": {
"Welcome": "स्वागत है",
"Hello": "नमस्कार",
Copy link

@saRvaGnyA saRvaGnyA Jan 7, 2021

Choose a reason for hiding this comment

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

"Hello": "नमस्ते",

Choose a reason for hiding this comment

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

नमस्कार is the action of greeting while नमस्ते is the greeting itself, so नमस्ते would seem more appropriate

"Connect": "कनेक्ट {{serviceauth}} अकाउंट",
"Unlink": "अनलिंक {{serviceauth}} अकाउंट",
"Login": "{{serviceauth}} से लोगिन करें",
"LogoARIA": "{{serviceauth}} प्रतीक चिन्ह"

Choose a reason for hiding this comment

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

"LogoARIA": "{{serviceauth}} प्र्‌‍‌तिक‍ चिन्ह"

"NewP5": "p5.js पर नये?",
"Report": "बग रिपोर्ट",
"Learn": "सीखें",
"Resources": "रीसॉर्सेज़",
Copy link

@saRvaGnyA saRvaGnyA Jan 7, 2021

Choose a reason for hiding this comment

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

"Resources": "साधन",

Choose a reason for hiding this comment

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

Phonetic translation for the technical term "Resources" was not matching. Here, a translation(like in the case of the word "Example") would be more appropriate.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was looking for a better word for this translation. Thanks.

},
"Toolbar": {
"Preview": "पूर्वावलोकन",
"Auto-refresh": "स्वयं रिफ़्रेश",
Copy link

@saRvaGnyA saRvaGnyA Jan 7, 2021

Choose a reason for hiding this comment

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

"Auto-refresh": "ऑटो-रिफ़्रेश",

"PlaySketchARIA": "स्केच चलाएं",
"PlayOnlyVisualSketchARIA": "केवल दृश्य स्केच चलाएं",
"StopSketchARIA": "स्केच बंद करे",
"EditSketchARIA": "स्केच नाम संपादित करें",
Copy link

@saRvaGnyA saRvaGnyA Jan 7, 2021

Choose a reason for hiding this comment

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

"EditSketchARIA": "स्केच का नाम संपादित करें",

@saRvaGnyA
Copy link

Respected Ma'am @catarak, I saw the call for review for this pull request on Processing Forums.
@abhieshekumar, terrific work on the translation buddy!

I viewed all the translations and most of them are correct. However, there are some improvements I wanted to suggest.
I've added relevant single-line comments to the required lines of the commits too.

A major improvement I would like to suggest is wherever you have translated the word "Stop" as "बंद", please change it to "रोके". बंद literally means "to close" while रोके means "to stop".

"SublimeText": "Sublime Text शॉर्टकट",
"CodeEditing": {
"Tidy": "साफ",
"FindText": "शब्द ढूंढना",
Copy link

@saRvaGnyA saRvaGnyA Jan 7, 2021

Choose a reason for hiding this comment

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

"FindText": "शब्द ढूंढे",

},
"General": {
"StartSketch": "स्केच शुरू करें",
"StopSketch": "स्केच बंद करो",

Choose a reason for hiding this comment

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

"StopSketch": "स्केच रोके",

},
"MobileIDEView":{
"Preferences": "प्राथमिकता",
"MyStuff": "मेरा स्टफ",

Choose a reason for hiding this comment

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

"MyStuff": "मेरे वस्तु",
मेरा स्टफ sounds pretty odd

"NoCollections": "कोई संग्रह नहीं"
},
"Editor": {
"OpenSketchARIA": "स्केच फ़ाइलों का नेविगेशन खोलें",

Choose a reason for hiding this comment

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

"OpenSketchARIA": "स्केच फ़ाइलों का नॅविगेशन खोलें",

},
"Editor": {
"OpenSketchARIA": "स्केच फ़ाइलों का नेविगेशन खोलें",
"CloseSketchARIA": "स्केच फ़ाइलों का नेविगेशन बंद करें",

Choose a reason for hiding this comment

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

"CloseSketchARIA": "स्केच फ़ाइलों का नॅविगेशन बंद करें",

"KeyUpLineNumber": "लाइन {{lineNumber}}"
},
"EditorAccessibility": {
"NoLintMessages": "कोई लिंट मैसिज नहीं ",

Choose a reason for hiding this comment

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

"NoLintMessages": "कोई लिंट मैसेज नहीं ",

},
"EmailVerificationView": {
"Title": "p5.js वेब एडिटर | ईमेल वेरीफिकेशन",
"Verify": "अपना ईमेल वेरफाइ करें",

Choose a reason for hiding this comment

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

"Verify": "अपना ईमेल व्हॅरीफाय करें",

@catarak
Copy link
Member

catarak commented Jan 7, 2021

Thanks for your suggestions @saRvaGnyA! @abhieshekumar you are welcome to accept these suggestions or decide they don't make sense to you, and let me know if I can support you in these decisions!

@abhieshekumar
Copy link
Contributor Author

abhieshekumar commented Jan 27, 2021

@catarak I have made the suitable changes. Thanks @saRvaGnyA.

@catarak
Copy link
Member

catarak commented Jan 29, 2021

Thank you for making these changes! I'm going to go ahead and merge this ✨ To be deployed soon!

@catarak catarak merged commit d7224b8 into processing:develop Jan 29, 2021
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.

None yet

4 participants