SimpleArmory icon indicating copy to clipboard operation
SimpleArmory copied to clipboard

I18n

Open unconditional opened this issue 9 years ago • 6 comments

This adds a basic i18n/l10n support. The main goal was to make achi URLs localized, i.e. a user selects the XX language and all WowHead URLs turn from http://wowhead.com/... into http://XX.wowhead.com/... which also makes the achi popups localized. But that would be somewhat weird to let user choose language and only update URLs, thus the UI translation was made possible as well (header.html as an example).

unconditional avatar Jun 17 '16 16:06 unconditional

This is a really good first start. This has been requested in the past, just haven't had cycles to go an implement it, so thanks for giving it a shot.

Couple of comments:

  1. I'm not sure I like the language selection as part of the header menu. I'd prefer it either be a drop down off the user/picture, or a submenu in help. Submenu in help would be easier I think. Though a drop down off the character/user would provide us more plugin points for future features.
  2. Seems like it only includes translations for the header menus and not translations in the pages themselves. Is this expected? I see wowhead links work, but the pages themselves and categories are still in english.

-kevin

kevinclement avatar Jun 21 '16 17:06 kevinclement

Hey, thanks for the feedback.

  1. Do you have any vision of what that language dropdown might look like attached to the profile picture/name given that the profile name itself is a link to the battlenet profile page?
  2. Yes I just translated the header as an example to make sure it works. I'll push other content translation later.

unconditional avatar Jun 22 '16 10:06 unconditional

I imagine it will probably turn into a drop down with a few options. I think I'm going to have to play with it a bit and check-in the change to allow for it, then we can build this on top of it.

kevinclement avatar Jun 22 '16 17:06 kevinclement

Hello! Came here for that PR :) Any updates on it? I think basic preset like XX.wowhead.com is really good starting point and SimpleArmory really needs it. Example from my guild (RU server, a lot of guild mates speak only Russian): wow armory laggy as a shit and very unfriendly, but we cant give a link to perfect SimpleArmory, because of the used language.

PS: If you need help with translation to Russian, I can help

rakshazi avatar Feb 25 '19 08:02 rakshazi

Hi!

This PR would need to be rebased to be able to be reviewed.

After looking into it, I think it would be a better approach to use the i18n features of Angular directly: https://angular.io/guide/i18n

This would certainly be easier for new translations.

seirl avatar Mar 07 '19 09:03 seirl

I'm going to look into it, I don't think it should be that hard to use the Angular i18n and this pull-request as a basis for translation!

seirl avatar Mar 07 '19 09:03 seirl