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)
Tracking
()
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.
Updated•3 years ago
|
Iteration: 99.1 - Feb 7 - Feb 20 → 99.2 - Feb 21 - Mar 6
Assignee | ||
Comment 1•3 years ago
|
||
Pushed by dmosedale@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c9a7ccd7e07
switch pvt browsing promo close string, r=pdahiya,fluent-reviewers
Comment 3•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox99:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Comment 4•3 years ago
|
||
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.
Description
•