Closed Bug 1172195 Opened 9 years ago Closed 9 years ago

Unlocalized string on mozilla.org/firefox/desktop

Categories

(www.mozilla.org :: L10N, defect)

Product:
www.mozilla.org
Component:
L10N
Version:
Production
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: theo, Assigned: flod)

Details

We have an unlocalized string on https://www.mozilla.org/firefox/desktop/

"That’s why we’re proud to be voted the Most Trusted Internet Company for Privacy."

Introduced by https://github.com/mozilla/bedrock/commit/0f6ac108394efa9b492eb847f61fdb93fa0c3166#diff-ba76492eda61d9819f980db9f56dc7b4R68 ~10 days ago.
FYI, just fixed French in r142151
D'oh, just noticed there are two unlocalized strings:

* Based on Tom’s Hardware Web Browser Grand Prix XVI results.
Ok, there are actually moref strings. I did a quick review of the patch, here's a recap of what I found:


In desktop/index.lang:

That’s why we’re proud to be voted the <a href="{{ url1 }}" data-interaction="outbound link">Most Trusted Internet Company for Privacy.</a>

* Based on <a data-interaction="outbound link" href="{{ url1 }}">Tom’s Hardware Web Browser Grand Prix XVI</a> results.


In sync.lang

Open the <a class="menu ga-link" data-interaction="outbound link click" href="{{ url }}#w_a-handy-new-menu">menu</a> in the top right of Firefox and select “<strong>Sign in to Sync.</strong>”


In fast.lang:

Firefox outperformed other browsers in recent independent tests to be named <br class="wide-br">the <a href="{{ url1 }}" data-interaction="outbound link">“speed king.”</a>


That should be it.
Wow, that patch broke the world :-\

I'll go through the patch to double check your list and fix localizations.
Assignee: nobody → francesco.lodolo
Some fixes in r142164 rr142165 r142166

Still need to go through the patch to double check.
On webdashboard these "missing" strings are displayed without the data-interaction attribute. Shall this attribute used or shall it not?
(In reply to Michael Wolf from comment #6)
> On webdashboard these "missing" strings are displayed without the
> data-interaction attribute. Shall this attribute used or shall it not?

It was a temporary thing, it should be already fixed.
OK, thank you, meanwhile I read the message of Michael Köhler and your reply to this.
Checked the patch and couldn't find any other string. Thanks for the list Théo, that helped a lot.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.