[pt-BR] Firefox 34 review notes

RESOLVED FIXED

Status

Mozilla Localizations
pt-BR / Portuguese (Brazil)
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gueroJeff, Assigned: reuben)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

3 years ago
Hi Reuben and team,

I was just moving through the sign-off for Firefox 34 (currently in Beta) and noticed a number of things that need to be looked at for pt-BR Firefox.

I'm not sure if this is an issue with the Pootle exports, or if it was intentional, but there are a many previously translated strings that were reverted to English and many untranslated strings that are high impact (like Loop strings). Here's the list of files that need to be looked at. If you could take some time during the Brazil community meetup to review these, that would be great :-) The URL leads to the specific run of compare-locales, which is where you can see the specific affected strings.


browser/chrome/browser/browser.properties
browser/chrome/browser/devtools/netmonitor.dtd
browser/chrome/browser/devtools/netmonitor.properties
browser/chrome/browser/devtools/profiler.dtd
browser/chrome/browser/devtools/storage.properties
browser/chrome/browser/devtools/timeline.dtd
browser/chrome/browser/devtools/timeline.properties
browser/chrome/browser/devtools/webconsole.properties
browser/chrome/browser/downloads/downloads.dtd
browser/chrome/browser/downloads/downloads.properties
browser/chrome/browser/loop/loop.properties
browser/chrome/browser/places/places.properties
browser/chrome/browser/preferences/preferences.properties
browser/chrome/browser/shellservice.properties
browser/chrome/browser/translation.properties
dom/chrome/layout/css.properties
netwerk/necko.properties
security/manager/chrome/pipnss/nsserrors.properties
services/sync/sync.properties
toolkit/chrome/global/autocomplete.properties
toolkit/chrome/mozapps/extensions/extensions.properties
toolkit/chrome/mozapps/handling/handling.properties

Comment 1

3 years ago
Yeah, I found a bunch, and I think we should go through the diff over the weekend.
I couldn't see anything actually reverted.

We had those two examples of similar strings, where the old was translated and the new wasn't.

- resetProfile.resetButton.label     Restaurar o %S…
+ refreshProfile.resetButton.label   Refresh %S…

- shareImageCmd.label     Compartilhar esta imagem
+ shareImage.label     Share This Image

and its access keys. You may notice the entities aren't the same. For the first one, the translation may change.

With regards to the untranslated strings, yeah, Jefferson said on the list he was going to use the untranslated strings for the missing ones. Therefore it was intentional, although it is not the ideal. I though we were on a tighter deadline and we wouldn't have enough time to translate the missing ones.

With respect to what to do now, I thin k that Reuben can already take care of that.
Flags: needinfo?(reuben.bmo)
It looks like we had some other errors introduced by the Pootle migration. Sorry, it isn't that easy to spot the errors on the char by char diffs.
(Assignee)

Comment 4

3 years ago
Yeah, I have most of it fixed locally on a beta tree. Jeff, what's the process here? Can I just push to beta when I'm done?
Flags: needinfo?(reuben.bmo) → needinfo?(jbeatty)

Comment 5

3 years ago
Yes, please.
Flags: needinfo?(jbeatty)
(Reporter)

Comment 6

3 years ago
Reuben, please also make sure that you make those changes in Pootle for Aurora too, in order to avoid having those problems reverted in the other channel.
(Assignee)

Comment 7

3 years ago
https://hg.mozilla.org/releases/l10n/mozilla-beta/pt-BR/rev/8c2590518bcf

I'll make the changes in Pootle as soon as I have more than one GPRS worth of bandwidth :)

Happy new year!
(Assignee)

Comment 8

3 years ago
Forgot to close this.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.