After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 695673 - status/keyboard: Translate IBus IME name if possible
status/keyboard: Translate IBus IME name if possible
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-03-12 03:18 UTC by Daiki Ueno
Modified: 2013-08-12 16:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
status/keyboard: Translate IBus IME name if possible (1.61 KB, patch)
2013-03-12 03:18 UTC, Daiki Ueno
reviewed Details | Review
status/keyboard: Translate IBus IME name if possible (1.52 KB, patch)
2013-07-10 10:39 UTC, Daiki Ueno
committed Details | Review

Description Daiki Ueno 2013-03-12 03:18:04 UTC
It would be nice if IBus IME names are translated as well as XKB layout names.  The soon-to-be-released IBus 1.5.2 will have get_textdomain() method of IBusEngineDesc, which helps translate IME names in IME's own translation domain.  

See https://github.com/ibus/ibus/commit/42dd60fe
Comment 1 Daiki Ueno 2013-03-12 03:18:06 UTC
Created attachment 238657 [details] [review]
status/keyboard: Translate IBus IME name if possible
Comment 2 Matthias Clasen 2013-03-12 10:53:34 UTC
Review of attachment 238657 [details] [review]:

::: js/ui/status/keyboard.js
@@ +484,3 @@
+                            longName = Gettext.dgettext(textdomain, longName);
+                    }
+                    displayName = language + ' (' + longName + ')';

Not a new bug, but translating the name here makes it more obvious that foo + '(' + bar + ')' is not a proper way to construct a string to show in the  UI - there should be a translatable template like "%s (%s)"
Comment 3 Daiki Ueno 2013-07-10 10:39:29 UTC
Created attachment 248823 [details] [review]
status/keyboard: Translate IBus IME name if possible

--
Use string.format() to construct display name, and also removed IBus API check as IBus 1.5.2 was released while ago.  May I request to update the jhbuild module from 1.5.1 to 1.5.2?
Comment 4 Rui Matos 2013-08-09 15:52:52 UTC
Review of attachment 248823 [details] [review]:

Looks good. Thanks
Comment 5 Rui Matos 2013-08-09 15:56:19 UTC
(In reply to comment #3)
> May I request to update the jhbuild
> module from 1.5.1 to 1.5.2?

Anyone with commit access can do it, feel free to change it for the 3.10 moduleset.
Comment 6 Daiki Ueno 2013-08-12 16:48:16 UTC
Attachment 248823 [details] pushed as 974331b - status/keyboard: Translate IBus IME name if possible