Mark pseudolocale as runtime or buildtime in language selection list

RESOLVED FIXED

Status

Firefox OS
Gaia::L10n
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gandalf, Assigned: stas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

3 years ago
It would be useful to recognize if we're running build time or runtime pseudolocale.
(Assignee)

Comment 1

3 years ago
Created attachment 8568792 [details]
Use { } for runtime qps

Zibi,what do you think about this approach?  Runtime pseudolocales are surrounded with curly braces.

My reasoning is this:  we should distinguish runtime qps because they're different. Buildtime don't inflict any perf penalty and they work exactly the same as regular locales so they should be displayed in the same manner.

I chose { and } because of their connotation with development.
Assignee: nobody → stas
Status: NEW → ASSIGNED
Attachment #8568792 - Flags: feedback?(gandalf)
(Reporter)

Comment 2

3 years ago
Hmm, not sure if that they are meaningful to a developer.

Alternative idea: (rt) (bt) postfixes

I don't have a strong opinion, just looking for something self-explanatory.
(Assignee)

Comment 3

3 years ago
I think developers should just use buildtime whenever possible so I want to assume w don't need more information in the name in this case.  It's the runtime qps that are "weird" and in fact can be thought of as fallback when buildtime is not available, and I'd like to mark them as special.
(Reporter)

Comment 4

3 years ago
agree. Let's leave build-time as default with no special marking. For runtime, I'm not sure if {} or (rt) is better.

The advantage of {} is that you can mark every string with it (or with '*' postfix?) and see which one you're running without going to Languages panel.

(rt) is more commonly known as runtime.

Kevin, do you have a preference?
Flags: needinfo?(kgrandon)
I agree that this might be a bit confusing for us and third party developers. TBH - I'm not sure if marking them as rt/bt is any better as it still requires you to be quite intimate with the l10n system.

My preference would be to opt for longer, more readable text if possible. I'm not sure if the text in this list truncates or shrinks, but if we're short on space we could do something like:

Dynamic Accented
Dynamic Mirrored
Packaged Accented
Packaged Mirrored
Flags: needinfo?(kgrandon)
Really though I don't know if 'dynamic' is any better than 'runtime', so something like 'Runtime Accented' would probably work just as well. You could also add "Eng" to it if you want to keep the English part.

There are only two hard things in Computer Science: cache invalidation and naming things.
(Reporter)

Comment 7

3 years ago
I like Runtime Accented and Packaged Accented.
(Assignee)

Comment 8

3 years ago
Should we show the words "Runtime" and "Packaged" localized into the pseudolanguage?

Řŭŭƞŧīīḿḗḗ Ȧȧƈƈḗḗƞŧḗḗḓ
ᴚnuʇıɯǝ Wıɹɹoɹǝp

Ƥȧȧƈķȧȧɠḗḗḓ Ȧȧƈƈḗḗƞŧḗḗḓ
Ԁɐɔʞɐƃǝp Wıɹɹoɹǝp

or

Runtime Ȧȧƈƈḗḗƞŧḗḗḓ
Runtime Wıɹɹoɹǝp

Packaged Ȧȧƈƈḗḗƞŧḗḗḓ
Packaged Wıɹɹoɹǝp

Even if braces are a little bit cryptic they elegantly avoid this problem :)
(Assignee)

Comment 9

3 years ago
(In reply to Staś Małolepszy :stas from comment #8)
 
> Řŭŭƞŧīīḿḗḗ Ȧȧƈƈḗḗƞŧḗḗḓ
> ᴚnuʇıɯǝ Wıɹɹoɹǝp
> 
> Ƥȧȧƈķȧȧɠḗḗḓ Ȧȧƈƈḗḗƞŧḗḗḓ
> Ԁɐɔʞɐƃǝp Wıɹɹoɹǝp

Looks like Bugzilla removed LRO and PDF from the above.  But you get the idea.
I like the idea of having the entire name localized myself. I don't think it's too hard to read.
(Assignee)

Comment 11

3 years ago
Created attachment 8569099 [details]
Use Runtime/Packaged

Here's what it's going to look like.
Attachment #8568792 - Attachment is obsolete: true
Attachment #8568792 - Flags: feedback?(gandalf)
Attachment #8569099 - Flags: feedback?(kgrandon)
Attachment #8569099 - Flags: feedback?(gandalf)
Created attachment 8569114 [details] [review]
[gaia] stasm:1136368-buildtime-qps-names > mozilla-b2g:master
Comment on attachment 8569099 [details]
Use Runtime/Packaged

This works for me!
Attachment #8569099 - Flags: feedback?(kgrandon) → feedback+
(Assignee)

Comment 14

3 years ago
Comment on attachment 8569114 [details] [review]
[gaia] stasm:1136368-buildtime-qps-names > mozilla-b2g:master

https://treeherder.mozilla.org/#/jobs?repo=gaia-try&revision=3cc1d5db3ce9
Attachment #8569114 - Flags: review?(gandalf)
(Reporter)

Comment 15

3 years ago
Comment on attachment 8569114 [details] [review]
[gaia] stasm:1136368-buildtime-qps-names > mozilla-b2g:master

lgtm
Attachment #8569114 - Flags: review?(gandalf) → review+
(Reporter)

Comment 16

3 years ago
Comment on attachment 8569099 [details]
Use Runtime/Packaged

wfm.

Since it's not building any dependency, I don't think we should sweat too much over the perfect name. Let's get this landed as it provides a valuable info to the user and if we'll come up with something better down the road, we can land the update.
Attachment #8569099 - Flags: feedback?(gandalf) → feedback+
You need to log in before you can comment on or make changes to this bug.