Bug 11551 - "done" and "space" are hardcoded in Keyboard.lua
: "done" and "space" are hardcoded in Keyboard.lua
Status: RESOLVED FIXED
Product: SB Touch
Classification: Unclassified
Component: UI
: unspecified
: PC Other
: -- normal (vote)
: MP
Assigned To: Software Localization Team
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-30 07:43 UTC by Ben Klaas
Modified: 2009-09-08 09:11 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments
patch to allow localized keyboard keys (3.67 KB, patch)
2009-03-31 06:56 UTC, Richard Titmuss
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Klaas 2009-03-30 07:43:13 UTC
unfortunate oversight: "done" and "space" are hard-coded in Keyboard.lua so will show up as such for any language.

Richard, is there any issue with having a jive.ui class query for currently selected language?

Michael, do you know if these strings can be poached from elsewhere? If so, do "done" or "space" take up 15 characters in Italian?
Comment 1 Michael Herger 2009-03-30 08:52:53 UTC
SLT - I guess these are two pretty common words. Would you have them in one of your memories already?
Comment 2 Ben Klaas 2009-03-30 16:02:29 UTC
assigning to Richard for comment on button text localization
Comment 3 Richard Titmuss 2009-03-31 02:14:44 UTC
Hmm, breaks the model as UI code is not meant to use strings!

So we have two options:
1. Everywhere the uses a keyboard needs to set the 'done' and 'space' strings into the keyboard widget.
2. Add a Framework:string() function that can use the global strings table.

The first option does not appeal, so it looks like Framework will need to allow access to the global strings table. The global strings in JiveMain should also be refactored to use this mechanism.

I may look at this later.
Comment 4 Richard Titmuss 2009-03-31 03:31:00 UTC
While fixin bug 11457 (changing '?' icon to 'help' label), it becomes clear that we need to be able to control if widgets are icons or text from the skin, and not the code. The skin should also be able to set a default text value of the widget. After this change this means the localized content would come from the skin. Just got to work this out, then the keyboard part of the fix should be easy.
Comment 5 Richard Titmuss 2009-03-31 06:56:09 UTC
Created attachment 5010 [details]
patch to allow localized keyboard keys
Comment 6 Richard Titmuss 2009-03-31 06:59:24 UTC
Patch for Ben to review. This adds localization for keyboard keys via the skin.
Comment 7 Richard Titmuss 2009-03-31 09:34:10 UTC
Patched checked in in r5040. Just the translations needed now.
Comment 8 Ben Klaas 2009-03-31 09:45:28 UTC
Michael, can you assist in getting us translations for these two strings?
Comment 9 Michael Herger 2009-04-01 03:19:30 UTC
Added a few more translations. Please note that there was already a KEYBOARD_DONE. I've copied those strings to SMALL_ENTER (which is being used as the "done" key).

Still missing "space". I added a German "Leerzeichen", because it's long enought to break potentially too small keys.

François - "Leerzeichen" does easily fit the space key in eg. the search dialog. Please still make sure translations for "space" aren't any longer than aobut 15 characters. Thanks!
Comment 10 Michael Herger 2009-04-08 03:11:44 UTC
François - any news?
Comment 11 Ben Klaas 2009-04-13 10:45:20 UTC
SLT Team- there are currently four bugs assigned to you that have a target of "MP". The deadline for that milestone is this Wednesday April 15th. Could you update us on whether we can expect updated translations on these bugs by that date?
Comment 12 Michael Herger 2009-04-14 02:17:56 UTC
change 5295 - done. Thanks!
Comment 13 Ben Klaas 2009-04-14 14:49:49 UTC
I think we're looking good here. The biggest potential issue is FR for 'done', but it fits nicely.

FI for 'space' is amazingly long, but we have lots of "space for space", so that's a non-issue.

closing.