Bug 1577465 - Protection report string revisions. r=fluent-reviewers,johannh,flod

Differential Revision: https://phabricator.services.mozilla.com/D44024

--HG--
extra : moz-landing-system : lando

browser/components/protections/content/lockwise-card.js

@@ -147,7 +147,7 @@ export default class LockwiseCard {
       );
       textEl.setAttribute("data-l10n-id", "lockwise-sync-status");
     } else {
-      textEl.setAttribute("data-l10n-id", "lockwise-sync-not-syncing");
+      textEl.setAttribute("data-l10n-id", "lockwise-sync-not-syncing-devices");
     }
     // Display the link for enabling sync if no synced devices are detected.
     if (syncedDevices === 0) {

browser/components/protections/content/monitor-card.js

@@ -75,7 +75,7 @@ export default class MonitorClass {
       monitorCard.classList.add("has-logins");
       headerContent.setAttribute(
         "data-l10n-id",
-        "monitor-header-content-logged-in"
+        "monitor-header-content-signed-in"
       );
       this.renderContentForUserWithLogins(monitorData);
     } else {
@@ -85,7 +85,10 @@ export default class MonitorClass {
       );
       signUpForMonitorLink.href = this.buildMonitorUrl(monitorData.userEmail);
       signUpForMonitorLink.setAttribute("data-l10n-id", "monitor-sign-up");
-      headerContent.setAttribute("data-l10n-id", "monitor-header-content");
+      headerContent.setAttribute(
+        "data-l10n-id",
+        "monitor-header-content-no-account"
+      );
       signUpForMonitorLink.addEventListener("click", () => {
         this.doc.sendTelemetryEvent("click", "mtr_signup_button");
       });
@@ -138,7 +141,7 @@ export default class MonitorClass {
     );
     infoMonitoredAddresses.setAttribute(
       "data-l10n-id",
-      "info-monitored-addresses"
+      "info-monitored-emails"
     );
 
     const infoKnownBreaches = this.doc.getElementById("info-known-breaches");
@@ -146,7 +149,7 @@ export default class MonitorClass {
       "data-l10n-args",
       JSON.stringify({ count: monitorData.numBreaches })
     );
-    infoKnownBreaches.setAttribute("data-l10n-id", "info-known-breaches");
+    infoKnownBreaches.setAttribute("data-l10n-id", "info-known-breaches-found");
 
     const infoExposedPasswords = this.doc.getElementById(
       "info-exposed-passwords"
@@ -155,7 +158,10 @@ export default class MonitorClass {
       "data-l10n-args",
       JSON.stringify({ count: monitorData.passwords })
     );
-    infoExposedPasswords.setAttribute("data-l10n-id", "info-exposed-passwords");
+    infoExposedPasswords.setAttribute(
+      "data-l10n-id",
+      "info-exposed-passwords-found"
+    );
 
     // Display Lockwise section if there are any potential breached logins to report.
     if (monitorData.potentiallyBreachedLogins > 0) {

browser/components/protections/content/protections.html

@@ -85,7 +85,7 @@
                 </div>
                 <div id="tracker" class="tab-content">
                   <p id="trackerTitle" class="content-title" data-l10n-id="tracker-tab-title"></p>
-                  <p id="trackerContent" data-l10n-id="tracker-tab-content">
+                  <p id="trackerContent" data-l10n-id="tracker-tab-description">
                     <a target="_blank" id="tracker-link" data-l10n-name="learn-more-link"></a>
                   </p>
                 </div>

browser/components/protections/content/protections.js

@@ -81,7 +81,7 @@ document.addEventListener("DOMContentLoaded", e => {
       "data-l10n-args",
       JSON.stringify({ count: data.sumEvents, earliestDate: dateInMS })
     );
-    summary.setAttribute("data-l10n-id", "graph-total-summary");
+    summary.setAttribute("data-l10n-id", "graph-total-tracker-summary");
 
     // Set a default top size for the height of the graph bars so that small
     // numbers don't fill the whole graph.

browser/components/protections/test/browser/browser_protections_lockwise.js

@@ -124,7 +124,7 @@ add_task(async function() {
     );
     is(
       syncedDevicesStatusText.getAttribute("data-l10n-id"),
-      "lockwise-sync-not-syncing",
+      "lockwise-sync-not-syncing-devices",
       "Not syncing to other devices."
     );
 

browser/components/protections/test/browser/browser_protections_monitor.js

@@ -93,7 +93,7 @@ add_task(async function() {
     );
     is(
       hasLoginsHeaderContent.textContent,
-      "Firefox Monitor warns you if your info has appeared in a known data breach",
+      "Firefox Monitor warns you if your info has appeared in a known data breach.",
       "Header content for user with monitor data is correct"
     );
 
@@ -213,12 +213,7 @@ async function checkNoLoginsContentIsDisplayed(tab, expectedLinkContent) {
     );
     is(
       noLoginsHeaderContent.getAttribute("data-l10n-id"),
-      "monitor-header-content",
-      "Header content for user with no logins is correct"
-    );
-    is(
-      noLoginsHeaderContent.getAttribute("data-l10n-id"),
-      "monitor-header-content",
+      "monitor-header-content-no-account",
       "Header content for user with no logins is correct"
     );
   });

browser/locales/en-US/browser/protections.ftl

@@ -6,7 +6,7 @@
 #   $count (Number) - Number of tracking events blocked.
 graph-week-summary =
   { $count ->
-     [one] { -brand-short-name } blocked  { $count } tracker over the past week
+     [one] { -brand-short-name } blocked { $count } tracker over the past week
     *[other] { -brand-short-name } blocked { $count } trackers over the past week
   }
 
@@ -14,10 +14,10 @@ graph-week-summary =
 #   $count (Number) - Number of tracking events blocked.
 #   $earliestDate (Number) - Unix timestamp in ms, representing a date. The
 # earliest date recorded in the database.
-graph-total-summary =
+graph-total-tracker-summary =
   { $count ->
-     [one] { $count } tracker blocked since { DATETIME($earliestDate, day: "numeric", month: "long", year: "numeric") }
-    *[other] { $count } trackers blocked since { DATETIME($earliestDate, day: "numeric", month: "long", year: "numeric") }
+     [one] <b>{ $count }</b> tracker blocked since { DATETIME($earliestDate, day: "numeric", month: "long", year: "numeric") }
+    *[other] <b>{ $count }</b> trackers blocked since { DATETIME($earliestDate, day: "numeric", month: "long", year: "numeric") }
   }
 
 # The terminology used to refer to categories of Content Blocking is also used in chrome/browser/browser.properties and should be translated consistently.
@@ -46,7 +46,7 @@ cookie-tab-title = Cross-Site Tracking Cookies
 cookie-tab-content = These cookies follow you from site to site to gather data about what you do online. They are set by third parties such as advertisers and analytics companies. Blocking cross-site tracking cookies reduces the number of ads that follow you around. <a data-l10n-name="learn-more-link">Learn more</a>
 
 tracker-tab-title = Tracking Content
-tracker-tab-content = Websites may load external ads, videos, and other content that contain tracking code. Blocking tracking content can help sites load faster, but some buttons, forms, and login fields might not work. <a data-l10n-name="learn-more-link">Learn more</a>
+tracker-tab-description = Websites may load external ads, videos, and other content with tracking code. Blocking tracking content can help sites load faster, but some buttons, forms, and login fields might not work. <a data-l10n-name="learn-more-link">Learn more</a>
 
 fingerprinter-tab-title = Fingerprinters
 fingerprinter-tab-content = Fingerprinters collect settings from your browser and computer to create a profile of you. Using this digital fingerprint, they can track you across different websites. <a data-l10n-name="learn-more-link">Learn more</a>
@@ -82,40 +82,40 @@ lockwise-sync-status =
      [one] Syncing to { $count } other device
     *[other] Syncing to { $count } other devices
   }
-lockwise-sync-not-syncing = Not syncing to other devices.
+lockwise-sync-not-syncing-devices = Not syncing to other devices
 
 monitor-title = Look out for data breaches
 monitor-link = How it works
-monitor-header-content = Check { -monitor-brand-name } to see if you’ve been part of a data breach and get alerts about new breaches.
-monitor-header-content-logged-in = { -monitor-brand-name } warns you if your info has appeared in a known data breach
+monitor-header-content-no-account = Check { -monitor-brand-name } to see if you’ve been part of a known data breach, and get alerts about new breaches.
+monitor-header-content-signed-in = { -monitor-brand-name } warns you if your info has appeared in a known data breach.
 monitor-sign-up = Sign Up for Breach Alerts
 auto-scan = Automatically scanned today
 
 # This string is displayed after a large numeral that indicates the total number
 # of email addresses being monitored. Don’t add $count to
 # your localization, because it would result in the number showing twice.
-info-monitored-addresses =
+info-monitored-emails =
   { $count ->
-     [one] Email address being monitored.
-    *[other] Email addresses being monitored.
+     [one] Email address being monitored
+    *[other] Email addresses being monitored
   }
 
 # This string is displayed after a large numeral that indicates the total number
 # of known data breaches. Don’t add $count to
 # your localization, because it would result in the number showing twice.
-info-known-breaches =
+info-known-breaches-found =
   { $count ->
-     [one] Known data breach has exposed your information.
-    *[other] Known data breaches have exposed your information.
+     [one] Known data breach has exposed your information
+    *[other] Known data breaches have exposed your information
   }
 
 # This string is displayed after a large numeral that indicates the total number
 # of exposed passwords. Don’t add $count to
 # your localization, because it would result in the number showing twice.
-info-exposed-passwords =
+info-exposed-passwords-found =
   { $count ->
-     [one] Password exposed across all breaches.
-    *[other] Passwords exposed across all breaches.
+     [one] Password exposed across all breaches
+    *[other] Passwords exposed across all breaches
   }
 
 full-report-link = View full report on <a data-l10n-name="monitor-inline-link">{ -monitor-brand-name }</a>