All users were logged out of Bugzilla on October 13th, 2018

Fix strings for Preferences tracking protection section (re-org v2)

VERIFIED FIXED in Firefox 56

Status

()

P1
normal
VERIFIED FIXED
a year ago
a year ago

People

(Reporter: flod, Assigned: rickychien)

Tracking

Trunk
Firefox 56
Points:
---
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox56 fixed, firefox57 verified)

Details

(Whiteboard: [photon-preference])

Attachments

(1 attachment)

(Reporter)

Description

a year ago
Splitting into a different bug at this point.

> -<!ENTITY  trackingProtection.description       "Tracking is when companies collect information about you to build a profile and display content based on your browsing and personal data.">
> -<!ENTITY  trackingProtection.radioGroupLabel   "Block known tracking companies from displaying content">
> +<!ENTITY  trackingProtection.description       "Tracking is the collection of your browsing data across multiple websites. Tracking can be used to build a profile and display content based on your browsing and personal information.">
> +<!ENTITY  trackingProtection.radioGroupLabel   "Use Tracking Protection to block known trackers">

These string IDs should have been changed.

Please also add an explanation on why we have trackingProtectionPBM5.label and trackingProtectionPBM6.label, because it's all but clear from the localization file if you don't have access to the mock-ups.
(Reporter)

Updated

a year ago
Flags: needinfo?(rchien)
(Assignee)

Updated

a year ago
Status: NEW → ASSIGNED
Flags: needinfo?(rchien)
Priority: -- → P1
Whiteboard: [photon-preference] → [photon-preference][triage]
Comment hidden (mozreview-request)
Not sure whether it is ok to add a comment in privacy.dtd. flod, could you take a look? thanks
(Reporter)

Comment 3

a year ago
mozreview-review
Comment on attachment 8881431 [details]
Bug 1376420 - Fix strings for Preferences tracking protection section

https://reviewboard.mozilla.org/r/152578/#review157764

This is missing the most important part (rename the changed strings)

::: browser/locales/en-US/chrome/browser/preferences/privacy.dtd:18
(Diff revision 1)
>  <!ENTITY  trackingProtectionNever.accesskey    "n">
>  <!ENTITY  trackingProtectionLearnMore.label    "Learn more">
>  <!ENTITY  trackingProtectionExceptions.label   "Exceptions…">
>  <!ENTITY  trackingProtectionExceptions.accesskey "x">
>  
> +<!-- LOCALIZATION NOTE (trackingProtectionPBM5.label): use for privacy.trackingprotection.ui.enabled = true -->

This string is displayed if privacy.trackingprotection.ui.enabled is set to true. This current happens on the release and beta channel.

(Assuming this happens also on Beta by default)

::: browser/locales/en-US/chrome/browser/preferences/privacy.dtd:21
(Diff revision 1)
>  <!ENTITY  trackingProtectionExceptions.accesskey "x">
>  
> +<!-- LOCALIZATION NOTE (trackingProtectionPBM5.label): use for privacy.trackingprotection.ui.enabled = true -->
>  <!ENTITY trackingProtectionPBM5.label         "Use Tracking Protection in Private Windows">
>  <!ENTITY trackingProtectionPBM5.accesskey     "v">
> +<!-- LOCALIZATION NOTE (trackingProtectionPBM6.label): use for privacy.trackingprotection.ui.enabled = false -->

This string is displayed if privacy.trackingprotection.ui.enabled is set to false. This current happens on the nightly channel.
Attachment #8881431 - Flags: review?(francesco.lodolo) → review-
(Reporter)

Comment 4

a year ago
This current happens -> This currently happens

Updated

a year ago
Flags: qe-verify+
QA Contact: hani.yacoub
Target Milestone: --- → Firefox 56
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Updated

a year ago
Whiteboard: [photon-preference][triage] → [photon-preference]
(Reporter)

Comment 7

a year ago
(In reply to Francesco Lodolo [:flod] from comment #0)
> Splitting into a different bug at this point.
> 
> > -<!ENTITY  trackingProtection.description       "Tracking is when companies collect information about you to build a profile and display content based on your browsing and personal data.">
> > -<!ENTITY  trackingProtection.radioGroupLabel   "Block known tracking companies from displaying content">
> > +<!ENTITY  trackingProtection.description       "Tracking is the collection of your browsing data across multiple websites. Tracking can be used to build a profile and display content based on your browsing and personal information.">
> > +<!ENTITY  trackingProtection.radioGroupLabel   "Use Tracking Protection to block known trackers">
> 
> These string IDs should have been changed.

Still not addressing this issue
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings

You need to use a different string ID, you changed these strings in bug 1370190
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
flod, sorry for not paying attention for that string changes. Can you review that patch? thanks!
Flags: needinfo?(francesco.lodolo)
(Reporter)

Comment 11

a year ago
mozreview-review
Comment on attachment 8881431 [details]
Bug 1376420 - Fix strings for Preferences tracking protection section

https://reviewboard.mozilla.org/r/152578/#review158332
Attachment #8881431 - Flags: review?(francesco.lodolo) → review+

Comment 12

a year ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a9ce3c904ec
Fix strings for Preferences tracking protection section r=flod

Comment 13

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7a9ce3c904ec
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
(Assignee)

Updated

a year ago
Flags: needinfo?(francesco.lodolo)

Comment 14

a year ago
Hi,
I'm not sure what should I verify in this bug, could anybody please provide us with some steps to reproduce?
Thanks.
Flags: needinfo?(francesco.lodolo)
(Reporter)

Comment 15

a year ago
(In reply to Hani Yacoub from comment #14)
> Hi,
> I'm not sure what should I verify in this bug, could anybody please provide
> us with some steps to reproduce?
> Thanks.

Not the bug assignee, but just confirm that these two strings are still there

> +<!ENTITY  trackingProtection.description       "Tracking is the collection of your browsing data across multiple websites. Tracking can be used to build a profile and display content based on your browsing and personal information.">
> +<!ENTITY  trackingProtection.radioGroupLabel   "Use Tracking Protection to block known trackers">
Flags: needinfo?(francesco.lodolo)

Comment 16

a year ago
Build ID: 20170820100343
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0

Verified as fixed on Firefox Nightly 57.0a1 on Windows 10 x 64, Mac OS X 10.12 and Ubuntu 16.04 x64.
Status: RESOLVED → VERIFIED
status-firefox57: --- → verified
You need to log in before you can comment on or make changes to this bug.