Closed Bug 1754115 Opened 3 years ago Closed 3 years ago

create new private browsing string_id so we can get rid of "Dismiss" in favor of "Close"

Categories

(Firefox :: Messaging System, enhancement, P1)

Firefox 98
enhancement

Tracking

()

VERIFIED FIXED
99 Branch
Iteration:
99.2 - Feb 21 - Mar 6
Tracking Status
firefox99 --- verified

People

(Reporter: dmosedale, Assigned: dmosedale)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

For the en-US only VPN experiment for 98, we've temporarily hard-coded the title of the "Close" button. The comment I added to the source:

  <!-- XXX The tooltip for the close button is intended to change to "Close" (it is currently
    -- "Dismiss"). The string change missed 98 Nightly; however, this UI is only surfaced in
    -- an experiment, and the experiment for 98 will be US-only.  So for now, we're
    -- hard-coding this string in the "title" attribute.
    --
    -- Bug 1754115 is already filed to do the right thing for 99, which is to get rid of this
    -- workaround, and replace the old string_id in 99 Nightly with a new one.  The intent
    -- is to land it on 99 Nightly right after this patch lands and is uplifted to 98. -->
  <button id="dismiss-btn" class="promo-dismiss" title="Close"></button>

Once that patch lands on mozilla-central 99, and is uplifted to 98, the next step is to fix
99 to do the right thing by giving that new string.

Iteration: 99.1 - Feb 7 - Feb 20 → 99.2 - Feb 21 - Mar 6
Pushed by dmosedale@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7c9a7ccd7e07 switch pvt browsing promo close string, r=pdahiya,fluent-reviewers
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Depends on: 1748240

I‘ve verified this enhancement using Firefox Beta 99.0b4 (Build ID: 20220315185755) on Windows 10 x64, macOS 11.6, and Ubuntu 20.04.

  • The tooltip shown when hovering the “x” button from the Promo messages on Private Windows contains the “Close” string.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: