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

Enable applying Apple Glossary #66

Closed
wants to merge 10 commits into from
Closed

Conversation

flemingm
Copy link
Contributor

@flemingm flemingm commented Nov 17, 2019

Using existing Glossary to do install translation. Added new menu item to File Menu to load and apply Apple Glot (.LG) files. issue #65

Copy link
Owner

@igorkulman igorkulman left a comment

Choose a reason for hiding this comment

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

I am afraid I have no idea what this PR is trying to achieve. Googling "Apple Glossary" did not give me any useful information. I can only guess this is supposed to be some kind of machine translation of missing values which I do not really like. I have seen to many apps with crappy localization exactly because of a feature like this.

Also the PR does not even build, looks like Japanese translation has been added to the project but the actual file is missing. No idea what this PR has to do with Japanese translation.

@igorkulman igorkulman self-assigned this Nov 19, 2019
@flemingm
Copy link
Contributor Author

You can find details about "Apple Glossary" at the bottom of:
https://developer.apple.com/internationalization/
(this is one of the bookmark I added to help menu).

This include all apple OS And application localization for all apple supported languages.

I agree machine or lookup translation is not perfect but when working with translators it help as a starting point.

Screen Shot 2019-11-20 at 12 40 43 AM

Note: Apple ID need to access them.

@flemingm
Copy link
Contributor Author

add missing JA files, sorry.

@igorkulman
Copy link
Owner

I looked into this again and I just confirmed my initial opinion that this functionality is not a good match for the editor.

I know you invested a lot of effort to this, I am sorry for that.

@igorkulman igorkulman closed this Dec 3, 2019
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