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

Set default browser first_run dialog on macOS and Linux #12679

Closed
karenkliu opened this issue Nov 12, 2020 · 9 comments · Fixed by brave/brave-core#7700
Closed

Set default browser first_run dialog on macOS and Linux #12679

karenkliu opened this issue Nov 12, 2020 · 9 comments · Fixed by brave/brave-core#7700
Assignees
Labels
needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop OS/Linux OS/macOS priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA/Yes release-notes/include

Comments

@karenkliu
Copy link

karenkliu commented Nov 12, 2020

Description

We need to update the onboarding flow with:

  • More aggressive pitch for set default browser for increased retention and engagement

Design for MacOS

Enable the first_run dialog that comes with Chromium. This dialog shows before the browser is launched. It comes with checkboxes for set default browser and send crash reports. Change the text to say 'private product analytics'.
image

Follow-up for these designs in this issue: #14183

Assets

Figma: https://www.figma.com/file/MFHpcXJMg2RUQpUJMAZZ4R/?node-id=146%3A0

@rebron rebron changed the title [MacOS] Set default browser, antifraud referral confirmation check, welcome experience updates set default browser, antifraud referral confirmation check, welcome experience updates Nov 16, 2020
@rebron rebron added QA/Yes release-notes/include priority/P3 The next thing for us to work on. It'll ride the trains. needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. labels Nov 16, 2020
@karenkliu karenkliu changed the title set default browser, antifraud referral confirmation check, welcome experience updates set default browser, welcome experience updates Nov 18, 2020
@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Jan 11, 2021
@rebron rebron changed the title set default browser, welcome experience updates Set default browser, welcome experience updates Jan 11, 2021
@karenkliu
Copy link
Author

Removed Yahoo from the search engine slide.

@karenkliu karenkliu changed the title Set default browser, welcome experience updates Set default browser first_run dialog on MacOS Jan 27, 2021
@rebron rebron changed the title Set default browser first_run dialog on MacOS Set default browser first_run dialog on macOS Jan 27, 2021
simonhong added a commit to brave/brave-core that referenced this issue Jan 27, 2021
@karenkliu karenkliu changed the title Set default browser first_run dialog on macOS Set default browser first_run dialog on macOS and Linux Jan 28, 2021
@btlechowski
Copy link

btlechowski commented Feb 15, 2021

Verification passed on beta channel

Brave 1.21.47 Chromium: 88.0.4324.152 (Official Build) beta (64-bit)
Revision 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS Linux

Verified first run dialog is shown
image
Verified on beta channel that the beta build was made default browser when Set Brave as your default browser is enabled
Verified default browser was not changed when Set Brave as your default browser is disabled
Verified automatic crash reporting is enabled when Help improve Brave... is enabled
Verified PPA is enabled when Help improve Brave... is enabled.
image
Verified automatic crash reporting is disabled when Help improve Brave... is disabled
Verified PPA is disabled when Help improve Brave... is disabled. Logged #14160
image

Verification passed on dev channel

Brave 1.21.52 Chromium: 88.0.4324.152 (Official Build) dev (64-bit)
Revision 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS Ubuntu 18.04 LTS

Verified first run dialog is shown
image
Verified on dev channel that the dev build was made default browser when Set Brave as your default browser is enabled
Verified default browser was not changed when Set Brave as your default browser is disabled
Verified automatic crash reporting is enabled when Help improve Brave... is enabled
Verified PPA is enabled when Help improve Brave... is enabled.
image
Verified automatic crash reporting is disabled when Help improve Brave... is disabled
Verified PPA is disabled when Help improve Brave... is disabled. Logged #14160
image

Verification passed on nightly channel

Brave 1.22.23 Chromium: 89.0.4389.48 (Official Build) nightly (64-bit)
Revision 0fe3c4589a6cf5ce719d167834dfa9cd8978937a-refs/branch-heads/4389@{#873}
OS Ubuntu 18.04 LTS

Verified first run dialog is shown
image
Verified on nightly channel that the nightly build was made default browser when Set Brave as your default browser is enabled
Verified default browser was not changed when Set Brave as your default browser is disabled
Verified automatic crash reporting is enabled when Help improve Brave... is enabled
Verified PPA is enabled when Help improve Brave... is enabled.
image
Verified automatic crash reporting is disabled when Help improve Brave... is disabled
Verified PPA is disabled when Help improve Brave... is disabled. Logged #14160
image

Verification passed on release channel

Brave 1.21.68 Chromium: 89.0.4389.58 (Official Build) (64-bit)
Revision 1a139f28ecc27719439e37c6b1533cee999cb802-refs/branch-heads/4389@{#1134}
OS Ubuntu 18.04 LTS

Verified first run dialog is shown
image
Verified on release channel that the release build was made default browser when Set Brave as your default browser is enabled
Verified default browser was not changed when Set Brave as your default browser is disabled
Verified automatic crash reporting is enabled when Help improve Brave... is enabled
Verified automatic crash reporting is disabled when Help improve Brave... is disabled


Verified on the following channels:

Brave	1.22.27 Chromium: 89.0.4389.48 (Official Build) nightly (x86_64)
Revision	0fe3c4589a6cf5ce719d167834dfa9cd8978937a-refs/branch-heads/4389@{#873}
OS	macOS Version 10.15.7 (Build 19H512)
Nightly (x64/intel)

Verified test plan from brave/brave-core#7700.

Confirmed first run dialog is shown:

Nightly

Confirmed when using Nightly channel that I was prompted about changing my default browser when "Set Brave as your default browser" was selected:

Nightly-OSprompt

Confirmed that Nightly build was made default browser when selected on OS-level prompt.
Confirmed automatic crash reporting is enabled when "Help improve Brave..." is selected.
Confirmed P3A is enabled when Help improve Brave... is selected.

Example Example
Nightly-default Nightly-enabled

Confirmed I was not prompted about changing my default browser when "Set Brave as your default browser" is not selected.
Confirmed automatic crash reporting is disabled when Help improve Brave... is not selected.
Confirmed P3A is disabled when Help improve Brave... is not selected.

Example Example Example
Nightly-disabled1 Nightly-disabled2 Nightly-disabled3
Brave	1.21.56 Chromium: 88.0.4324.152 (Official Build) dev (x86_64)
Revision	6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS	macOS Version 10.15.7 (Build 19H512)
Dev (x64/intel)

Verified test plan from brave/brave-core#7700.

Confirmed first run dialog is shown:

Dev

Confirmed when using Dev channel that I was prompted about changing my default browser when "Set Brave as your default browser" was selected:

Dev-OSprompt

Confirmed that Dev build was made default browser when selected on OS-level prompt.
Confirmed automatic crash reporting is enabled when "Help improve Brave..." is selected.
Confirmed P3A is enabled when Help improve Brave... is selected.

Example Example
1 2

Confirmed I was not prompted about changing my default browser when "Set Brave as your default browser" is not selected.
Confirmed automatic crash reporting is disabled when Help improve Brave... is not selected.
Confirmed P3A is disabled when Help improve Brave... is not selected.

Example Example Example
3 4 5
Brave	1.21.55 Chromium: 88.0.4324.152 (Official Build) beta (x86_64)
Revision	6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS	macOS Version 10.15.7 (Build 19H512)
Beta (x64/intel)

Verified test plan from brave/brave-core#7700.

Confirmed first run dialog is shown:

Beta

Confirmed when using Beta channel that I was prompted about changing my default browser when "Set Brave as your default browser" was selected:

Beta-OSprompt

Confirmed that Beta build was made default browser when selected on OS-level prompt.
Confirmed automatic crash reporting is enabled when "Help improve Brave..." is selected.
Confirmed P3A is enabled when Help improve Brave... is selected.

Example Example
1 2

Confirmed I was not prompted about changing my default browser when "Set Brave as your default browser" is not selected.
Confirmed automatic crash reporting is disabled when Help improve Brave... is not selected.
Confirmed P3A is disabled when Help improve Brave... is not selected.

Example Example Example
3 4 5
Brave	1.21.55 Chromium: 88.0.4324.152 (Official Build) beta (arm64)
Revision	6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS	macOS Version 11.2 (Build 20D64)
Beta (arm64/M1)

Verified test plan from brave/brave-core#7700.

Confirmed first run dialog is shown:

Beta-M1

Confirmed when using Beta channel that I was prompted about changing my default browser when "Set Brave as your default browser" was selected:

Beta-OSpromptM1

Confirmed that Beta build was made default browser when selected on OS-level prompt.
Confirmed automatic crash reporting is enabled when "Help improve Brave..." is selected.
Confirmed P3A is enabled when Help improve Brave... is selected.

Example Example
1 2

Confirmed I was not prompted about changing my default browser when "Set Brave as your default browser" is not selected.
Confirmed automatic crash reporting is disabled when Help improve Brave... is not selected.
Confirmed P3A is disabled when Help improve Brave... is not selected.

Example Example Example
3 4 5

@btlechowski
Copy link

Set as QA/Blocked till we get a release build for 1.21.x

@btlechowski
Copy link

No longer QA/Blocked as we have a 1.21.x release build.

@stephendonner
Copy link

Verified FIXED using the testplan from brave/brave-core#7700 with:

Brave 1.21.68 Chromium: 89.0.4389.58 (Official Build) (x86_64)
Revision 1a139f28ecc27719439e37c6b1533cee999cb802-refs/branch-heads/4389@{#1134}
OS macOS Version 11.2.2 (Build 20D80)

Confirmed first run dialog is shown:

screenshot 109192237-d386a000-774b-11eb-91a1-c695b60ffd1e

Confirmed when using Release channel that I was prompted about changing my default browser when "Set Brave as your default browser" was selected:

screenshot Screen Shot 2021-02-25 at 1 54 40 PM

Confirmed that Release build was made default browser when selected on OS-level prompt.

Confirmed automatic crash reporting is enabled when "Help improve Brave..." is selected.

Confirmed P3A is enabled by default in brave://settings/privacy

screenshots Screen Shot 2021-02-25 at 1 59 55 PM

Confirmed I was not prompted about changing my default browser when Set Brave as your default browser is not selected.
Confirmed in the Default browser section inbrave://settings that the Make default button exists and is clickable.
Confirmed automatic crash reporting is disabled when Help improve Brave... is not selected.
Confirmed P3A remains enabled, by default, in brave://settings/privacy

screenshots Screen Shot 2021-02-25 at 2 03 09 PMScreen Shot 2021-02-25 at 2 09 54 PMScreen Shot 2021-02-25 at 2 03 25 PM

@pitsi
Copy link

pitsi commented Mar 3, 2021

I just upgraded to 1.21.x and I get the prompt ON EVERY LAUNCH. Shouldn't it be shown just once?
Debian testing/unstable x64 and x-www-browser (= the system wide setting for the default browser) is already set to brave

$ file /usr/bin/x-www-browser
/usr/bin/x-www-browser: symbolic link to /etc/alternatives/x-www-browser
$ file /etc/alternatives/x-www-browser
/etc/alternatives/x-www-browser: symbolic link to /usr/bin/brave-browser-stable

If there is a switch or a flag that can disable it, please tell me. Thank you in advance.

@matthieu-bt
Copy link

Same issue here, on Windows 10. I use Brave in private mode only. And since this morning, EVERY time I open it, I get the prompt.

@pitsi
Copy link

pitsi commented Mar 4, 2021

If someone openened a new issue report for that spammy prompt, let's continue our discussion there please.
I would open it, but I am too busy today with an rpi4 that... "misbehaves" :D
And that is the reason I have opened brave 2-3 times today, but the prompt spammed me only once.

@pitsi
Copy link

pitsi commented Mar 5, 2021

After a full day of opening and closing brave dozens of times I noticed that the issue no longer happens, at least on my end, so I won't open the report I was saying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop OS/Linux OS/macOS priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants