Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

PBM "Always launch in private mode" issues #5396

Closed
sblatz opened this issue Sep 17, 2019 · 11 comments
Closed

PBM "Always launch in private mode" issues #5396

sblatz opened this issue Sep 17, 2019 · 11 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. E3 Estimation Point: average, 2 - 3 days eng:qa:verified QA Verified Feature:PrivateBrowsing needs:product P1 Current sprint S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Milestone

Comments

@sblatz
Copy link
Contributor

sblatz commented Sep 17, 2019

Issues with the current implementation:

  1. Selecting to "always" launch in private mode does not actually set the app to always launch privately. The first link you open will be in private mode, but after that they'll launch in normal browsing mode.

always broken 2019-09-17 14_37_50

  1. Currently if a user selects that they "always" want to launch in private mode, they have no way of undoing this selection (same for always launching in normal mode) since the separate launching modes do not show up as separate apps in the default browser settings. We likely want a setting in app to handle this (e.g. a toggle under "Privacy" that says "Always launch in private mode")

image

┆Issue is synchronized with this Jira Task

@sblatz sblatz added 🐞 bug Crashes, Something isn't working, .. P1 Current sprint Feature:PrivateBrowsing S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist labels Sep 17, 2019
@sblatz sblatz added this to To be Triaged in Fenix Sprint Kanban via automation Sep 17, 2019
@sblatz sblatz moved this from To be Triaged to Sprint Backlog in Fenix Sprint Kanban Sep 17, 2019
@sblatz sblatz self-assigned this Sep 17, 2019
@sblatz
Copy link
Contributor Author

sblatz commented Sep 17, 2019

These fixes should land at the same time as fixing #4780

@sblatz
Copy link
Contributor Author

sblatz commented Sep 17, 2019

My recommendation here is to simply add a SwitchPreference in settings under Privacy to "Always launch in private mode."

The only worry would be users not "finding out" about it since the original design allowed users to find it when just opening a link.

sblatz added a commit to sblatz/fenix that referenced this issue Sep 17, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Sep 17, 2019
boek pushed a commit that referenced this issue Sep 18, 2019
boek pushed a commit that referenced this issue Sep 18, 2019
@lobontiumira
Copy link

Might be related to #5375 and #5131.

@sblatz
Copy link
Contributor Author

sblatz commented Sep 18, 2019

This is a demo of my proposed solution. Basically all external links would switch the user to private mode and then open the link.

@sblatz
Copy link
Contributor Author

sblatz commented Sep 18, 2019

cc @vesta0 and @topotropic for your thoughts on pulling this in to 2.1?

@sblatz
Copy link
Contributor Author

sblatz commented Sep 18, 2019

@jpfaller could you provide official strings for the "Open links in private mode"? I tried "Open external links in private mode" (which is a bit more accurate, but it takes up two lines and looks bad).

@sblatz sblatz added needs:strings Needs strings needs:product needs:UX-feedback Needs UX Feedback labels Sep 18, 2019
@jpfaller
Copy link

What about:

Launch links in a private tab

@sblatz sblatz moved this from Sprint Backlog to In Progress in Fenix Sprint Kanban Sep 18, 2019
@sblatz
Copy link
Contributor Author

sblatz commented Sep 19, 2019

Updated with string @jpfaller requested.

Touched base with @topotropic and her feedback was to just move the preference toggle down a slot which I have also done.

sblatz added a commit to sblatz/fenix that referenced this issue Sep 19, 2019
@sblatz sblatz added the E3 Estimation Point: average, 2 - 3 days label Sep 19, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Sep 19, 2019
@sblatz sblatz removed needs:UX-feedback Needs UX Feedback needs:strings Needs strings labels Sep 19, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Sep 19, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Sep 19, 2019
@sblatz sblatz moved this from In Progress to In Review in Fenix Sprint Kanban Sep 20, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Sep 20, 2019
@ekager ekager added the eng:qa:needed QA Needed label Sep 20, 2019
@sblatz sblatz added this to the v2.1 milestone Sep 20, 2019
@sblatz sblatz moved this from In Review to Ready for QA in Fenix Sprint Kanban Sep 20, 2019
@joebloggs7657

This comment has been minimized.

@sblatz
Copy link
Contributor Author

sblatz commented Sep 23, 2019

For QA: the new expected behavior here is:
0) Be in normal browsing mode

  1. Enable the toggle in settings
  2. Open a link from another app (with Fenix as your default browser)

Then: link opens in private mode

Also:
0) Be in normal browsing mode

  1. Disable the toggle in settings
  2. Open a link from another app (with Fenix as your default browser)

Then: link opens in normal mode

@lobontiumira
Copy link

Verified that when the toggle "Launch links in a private tab" is on, all the links opened through external apps are opened in private mode.
Tested with the latest Nightly build 190923 (#12661829) with Motorola Moto G6 (Android 8), Nokia 6 (Android 7.1.1), and Samsung Galaxy Tab A6 (Android 5.1).

Fenix Sprint Kanban automation moved this from Ready for QA to Done Sep 24, 2019
@lobontiumira lobontiumira added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Sep 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. E3 Estimation Point: average, 2 - 3 days eng:qa:verified QA Verified Feature:PrivateBrowsing needs:product P1 Current sprint S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
None yet
Development

No branches or pull requests

5 participants