767568 - Pt-PT has "Pair a Device" in the explanation in English

Status: VERIFIED FIXED

(Mozilla Localizations :: pt-PT / Portuguese, defect)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot

1. install 14 beta
2. set device to Pt-Pt (Samsung Galaxy Nexus)
3. launch fennec and go to Menu-> Settings-> More-> Sync (These are properly translated into PT-PT, just calling out the English because it's easier to understand)
Expected: "Pair a Device" should be in Pt-PT
Actual: It's in English; Desktop version is Translated.

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Noming for Beta N

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

To note, Pike and I spoke about this on irc, and he stated that it must have been there since 11 or so...  If it's been in there for that long, I don't think it's critical that we have it immediately, though I would like to fix it in the next release if possible.

Comment 3

[Alex Keybl [:akeybl]]

At this point, the risk vs reward of getting this worked out and re-spinning isn't worth it. I'd suggest .N+.

Comment 4

[Allison Naaktgeboren :ally]

based on comment 2, I'm strongly against re-spinning for this. I'd consider it as a ridealong if we had to respin for something else.

Comment 5

[Aaron Train [:aaronmt]]

Do not block. Ship it.

Comment 6

[Johnathan Nightingale [:johnath]]

So say we all. .N+

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Axel - Is this change in the l10n repo yet?

Comment 8

[Axel Hecht [:Pike]]

I'm not on the hook for this string, you can check yourself on http://hg.mozilla.org/releases/l10n/mozilla-beta/pt-PT/annotate/default/mobile/android/base/sync_strings.dtd#l10.

Also, making sure that Carlos gets the bugmail on this one.

Comment 9

[chris hofmann]

(In reply to Mark Finkle (:mfinkle) from comment #7)
> Axel - Is this change in the l10n repo yet?

Axel's not the right person to ask.  [e-mail] and maybe [e-mail] as back up are the module owners and folks that can help here.

Also looks like folks didn't read the recommendation in comment 2.  Its been this way  for quite some time and no one seems to have noticed.  We could check for user feedback in forums but at this point we only have around 11k users for all fennec pt-PT.    Seems like we could really just pick this up on the next cycle if needed.

Comment 10

[Carlos Simao]

corrected
for l10n-central http://hg.mozilla.org/l10n-central/pt-PT/rev/00ba34d890da
for aurora http://hg.mozilla.org/releases/l10n/mozilla-aurora/pt-PT/rev/7f08713ec7f1

Comment 11

[Axel Hecht [:Pike]]

Sweet, thanks Carlos. Do you have the opportunity to test this on a device on central or aurora? Otherwise, we can get someone from our QA folks to quickly drive a test and verify that stuff is sized right and such.

After that, it'd be awesome to get the fix transplanted to beta, too.

Comment 12

[Carlos Simao]

no chance to test it. My 2 android mobiles are a unsupported device.

Comment 13

[Axel Hecht [:Pike]]

Naoki, can you test tomorrow's nightly or aurora build again?

Comment 14

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Sure, Pike.  I'll put it on my to-do list for tomorrow.

Comment 15

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot of fix in Nightly 7/6/2012

Screenshot of the fix.  Not sure if it's correctly translated, but the translation is there (rather than the english)

Comment 16

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 2nd screenshot

Screenshot of where I originally saw the issue as well.

Comment 17

[Carlos Simao]

yes it's correctly translated

Comment 18

[Axel Hecht [:Pike]]

Sweet, Carlos, can you transplant the fix to beta and sign-off?

Comment 19

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

QAWanted still for checking Beta.

Comment 20

[Aaron Train [:aaronmt]]

Landed as 

http://hg.mozilla.org/releases/l10n/mozilla-beta/pt-PT/rev/dbfc087ff314

Comment 21

[Aaron Train [:aaronmt]]

Verified Fixed across the board; 14.0 Beta 12 (build #1), 15.0a2, 16.0a1