Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Work around welcome page disappearing post install #2798

Merged
merged 1 commit into from
Sep 22, 2021

Conversation

ghostwords
Copy link
Member

@ghostwords ghostwords commented Sep 3, 2021

This is meant to help Firefox users not lose the welcome page when it gets inadvertently closed when the user chooses to first interact with the post-install Private browsing permission hanger (a reasonable choice) and allows Privacy Badger to run in Private windows.

Related to #2781.

To test:

  1. Get Firefox Nightly
  2. Toggle xpinstall.signatures.required to false in about:config
  3. Create an XPI: cd src and then zip -r badger.xpi *
  4. Drag and drop the XPI into about:addons, switch to the Firefox window, click "Add" in the unverified extension prompt
  5. Check "Allow this extension to run in Private Windows", click "Okay"

If you interact with the welcome page first, finish reading it, and respond to the Private browsing permission hanger in under a minute, we are going to reopen the welcome page, which is a bit annoying.

We could shorten the timer duration, but then we run the opposite risk of not reopening the welcome page if the user took too long to interact with the permission hanger. Is one minute a good compromise?

@ghostwords
Copy link
Member Author

Related Mozilla bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1558336

@ghostwords
Copy link
Member Author

@ablanathtanalba Thoughts?

Copy link
Contributor

@ablanathtanalba ablanathtanalba left a comment

Choose a reason for hiding this comment

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

Looks good to me, just tested in the latest nightly on macOS Big Sur 11.6.

I think 1 minute is a good compromise, some thoughts though; it's about a minute to skim through the welcome page text (as it is now) if they interact with that first before the private windows hanger prompt. I haven't tested on the new welcome page drafts, but it does take more than a minute to read through the content now at a leisurely pace, so there's a small chance people could get be there reading first, get finished, then click the hanger prompt allowing it to run in private windows, and then they're jumped up to the top of the welcome page again. A little wonky, but worth the compromise imo.

@ghostwords
Copy link
Member Author

ghostwords commented Sep 22, 2021

Yeah, we're probably erring on the side of reopening the welcome page when we shouldn't. Hard to say without user testing, but this should be OK as reopening the welcome page when it's not wanted (at most once) is much less bad than having it close with no recourse.

ghostwords added a commit that referenced this pull request Sep 22, 2021
Work around welcome page disappearing post install.

Happens when the user allows Privacy Badger to run in Private windows
using the post-install Private browsing permission hanger in Firefox
before reviewing the welcome page.

Related Mozilla bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1558336
@ghostwords ghostwords merged commit 0a0ca5d into master Sep 22, 2021
@ghostwords ghostwords deleted the welcome-page-closing-workaround branch September 22, 2021 13:57
ghostwords added a commit that referenced this pull request Nov 23, 2021
@ghostwords
Copy link
Member Author

Note that there was an important followup fix to this workaround: 12fa27f

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