Skip to content

Bug 2048200: Expose longfox secret setting to all release types.#297

Closed
pollym wants to merge 1 commit into
mozilla-firefox:autolandfrom
pollym:expose-longfox-secret-setting
Closed

Bug 2048200: Expose longfox secret setting to all release types.#297
pollym wants to merge 1 commit into
mozilla-firefox:autolandfrom
pollym:expose-longfox-secret-setting

Conversation

@pollym

@pollym pollym commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This means that if people are in the longfox experiment and hate it, they can turn it off.
conversely, if you are longfox curious you could switch it on!

@github-actions

Copy link
Copy Markdown
Contributor

View this pull request in Lando to land it once approved.

@lando-web lando-web Bot requested a review from a team June 17, 2026 10:17
@pollym pollym force-pushed the expose-longfox-secret-setting branch 4 times, most recently from eaa30c4 to 514d3a2 Compare June 17, 2026 11:09
@mozilla-code-review

Copy link
Copy Markdown

No new issues detected. This pull request is 🆗

@pollym pollym force-pushed the expose-longfox-secret-setting branch from 514d3a2 to db14b2e Compare June 17, 2026 13:27
@mozilla-code-review

Copy link
Copy Markdown

No new issues detected. This pull request is 🆗

@pollym pollym force-pushed the expose-longfox-secret-setting branch from db14b2e to 674fe40 Compare June 17, 2026 15:48
@mozilla-code-review

Copy link
Copy Markdown

No new issues detected. This pull request is 🆗

@moyheen moyheen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good, thanks for making the changes!

…ine homepage switch message.

This means that if people are in the longfox experiment and hate it, they can turn it off and get the privacy dashboard.
(conversely, if you are longfox curious you could switch it on!)
Also on the homepage settings screen, if you have longfox enabled, change the text of the label of the privacy report button to match what the button says. This makes it easier for people who want to get rid of the button completely and are in the longfox experiment.
@pollym pollym force-pushed the expose-longfox-secret-setting branch from 674fe40 to 31cd6d8 Compare June 18, 2026 13:06
@pollym

pollym commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

behaviour tweaked slightly following chat with ux.

  • removed the longfox switch from the homepage settings, so there is only one switch.
    -- this switch shows / hides the button
    -- this switch has a label which matches the button type: either "privacy report" or, if longfox is enabled, "help catch trackers"

  • there is still an "enable longfox" switch in secret settings,
    -- this switch will change the behaviour of the button
    -- so qa can test the feature and if people are desperate to play the game but aren't in the experiment there is a way to do that. Or, if they are in the experiment but want the privacy dashboard, they can also flip this setting.

this is a much smaller simpler change :)
try is running here

@lando-worker

lando-worker Bot commented Jun 18, 2026

Copy link
Copy Markdown

Pull request closed by commit 44d22b4

lando-worker Bot pushed a commit that referenced this pull request Jun 18, 2026
…yheen

This means that if people are in the longfox experiment and hate it, they can turn it off.
conversely, if you are longfox curious you could switch it on!

Pull request: #297
@lando-worker lando-worker Bot closed this Jun 18, 2026
@mozilla-code-review

Copy link
Copy Markdown

No new issues detected. This pull request is 🆗

lando-worker Bot pushed a commit that referenced this pull request Jun 22, 2026
…scalc DONTBUILD

This means that if people are in the longfox experiment and hate it, they can turn it off.
conversely, if you are longfox curious you could switch it on!

Pull request: #297

Differential Revision: https://phabricator.services.mozilla.com/D307565
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants