Skip to content

Comments

i18n support and Japanese translation#76

Merged
Ealdwulf merged 12 commits intoKanoComputing:jessiefrom
japonophile:japanese_translation_jessie
Jun 20, 2016
Merged

i18n support and Japanese translation#76
Ealdwulf merged 12 commits intoKanoComputing:jessiefrom
japonophile:japanese_translation_jessie

Conversation

@japonophile
Copy link
Contributor

Added i18n initialization and implemented Japanese translation for kano-apps.
The UI is correctly displayed in Japanese, but each app description (fetched from .app files) is still in English only.

@alex5imon
Copy link

😍

@Ealdwulf
Copy link
Contributor

Just to let you know that we haven't forgotten about these - This one looks pretty safe to me. However when I tried it nothing came up in Japanese (I know the descriptions don't, but I didn't find any Japanese). It's probably something trivial, I'll have another look soon.

@japonophile
Copy link
Contributor Author

Great to hear back from you!

Here are the quick steps I follow to "japanify" my Kano:

$ sudo dpkg-reconfigure locales
Choose:
en_GB.UTF-8 UTF-8
ja_JP.EUC-JP EUC-JP
ja_JP.UTF-8 UTF-8 (*)

$ apt-get install ttf-kochi-gothic xfonts-intl-japanese xfonts-intl-japanese-big xfonts-kaname

I guess you may have missed that last step.
If the proper fonts are not available, you won't see anything.

@japonophile
Copy link
Contributor Author

Another thing I noticed, if you are installing kano-apps through setup.py, is that it will not copy the kano-apps script at the correct location. To test on my Kano, I just manually copied kano-apps to /usr/bin.

@Ealdwulf
Copy link
Contributor

Thanks @japonophile . The problem I was having was that the debian/install file was missing the po files: c44fd06
With that fixed it displays the Japanese fine. We are doing a release right now, should be able to merge this afterwards.

pkgs = " ".join(app["packages"])

cmd = "rxvt -title 'Uninstalling {}' -e bash -c ".format(app["title"])
cmd = ("rxvt -title " + _("'Uninstalling {}'") + " -e bash -c ").format(app["title"])
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a problem here, in that os.system expects ordinary strings, so we need to do os.system(cmd.encode('utf8')) below, and similarly elsewhere in this file. I've made this change locally.

@Ealdwulf
Copy link
Contributor

Ealdwulf commented Jun 17, 2016

Hi @japonophile , I'm going to formally close this one, but actually I've opened a new one which incorporates your commits here: #77
Thanks!

@Ealdwulf Ealdwulf merged commit d3f8819 into KanoComputing:jessie Jun 20, 2016
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.

3 participants