1057696 - [Settings] Browser Privacy: l10n issues

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Francesco Lodolo [:flod]]

https://github.com/mozilla-b2g/gaia/commit/3579256554985d6ad49bee4615651a255962f22f

1) data-l10n-id="clearBookmarksAppsData">

We don't have such an ID, we have clearBookmarkAppsData.

2) We need a different string for menu item and header in Settings (see across settings.properties for references). You created the reference in the header, but not added the string inside settings.properties

<h1 data-l10n-id="browserPrivacy-header"> Browser Privacy </h1>

3) The main title "Confirmation" is hard-coded. 
https://github.com/mozilla-b2g/gaia/commit/3579256554985d6ad49bee4615651a255962f22f#diff-43f348489f2f3323b5a6339221ad7998R34

Why are we not enforcing tests with pseudo-locales? They're build at runtime for this very reason, and all of these issues would have been found within seconds.

Comment 1

[Francesco Lodolo [:flod]]

Attachment: Search

I was going to file another bug but discovered it comes from the same bug :-\

Search is using data-l10n-id="search"

We have a "search-header", but not a "search" id
https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/locales/settings.en-US.properties

Comment 2

[Francesco Lodolo [:flod]]

Attachment: Pull request on Github

Fixing since we're approaching FL and string freeze.

* Added new clear-boomarks-data string for the button instead of reusing the existing one from another panel.
* Switched ID from "confirmation" to less generic "confirm-clear-title".

Comment 3

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8480529 [details] [review]
Pull request on Github

Pseudo locale should be used. I feel embarrassed by not catching this. :(

The patch looks good to me, thanks!

Comment 4

[Francesco Lodolo [:flod]]

Thanks Arthur, asking for checkin.

Comment 5

[Arthur Chen [:arthurcc][inactive after 6/30]]

master: e43e40ce50c1b6f331f3f00ef6cf954dff6abbc9