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 browser as default' dialog on 2nd and nth launch #12203

Closed
karenkliu opened this issue Oct 19, 2020 · 9 comments · Fixed by brave/brave-core#7716 or brave/brave-core#7772
Closed
Assignees

Comments

@karenkliu
Copy link

karenkliu commented Oct 19, 2020

Description

One of the issues made evident from lapsed user research is that we can do better in encouraging users to set Brave as their default browser.

Currently, the "set browser dialog" is displayed as a non-user-blocking infobar on 2nd launch.

Designs

Change the infobar to a user-blocking browser dialog to encourage interaction.

image

Brave will check if it has been set as the default browser on every launch. This dialog reappears:

  • If the user hits "not now", the dialog reappears on the 3rd, 4th, 8th, and 20th launch.
  • If the user had set Brave as the default, and then it was not the default on next launch (possibly due to updating OS), then show the dialog regardless of the nth launch

Dark theme is supported:
image

Assets

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

Note

This is a follow up issue to:

@karenkliu karenkliu added OS/Android Fixes related to Android browser functionality OS/Desktop labels Oct 19, 2020
@karenkliu karenkliu changed the title [Desktop] Include Set browser as default in the welcome experience [Desktop] More aggressive Set browser as default dialog on 2nd launch Nov 12, 2020
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Nov 18, 2020
@karenkliu
Copy link
Author

@rebron When do you want this to dialog to show again if the user selects "not now" ?

@karenkliu
Copy link
Author

@rebron @bsclifton For this issue, we need to finish defining the reappearance of the dialog. What are your thoughts on this cadence (and if it's feasible):

If the user hits "not now", the dialog reappears on the 3rd, 4th, 8th, and 20th launch.
If the user had set Brave as the default, and then it was not the default on next launch (possibly due to updating OS), then show the dialog regardless of the nth launch

@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 [Desktop] More aggressive Set browser as default dialog on 2nd launch [Desktop] Set browser as default dialog on 2nd launch Jan 11, 2021
@bsclifton
Copy link
Member

I like the frequency - I think it makes sense to not reset the count if browser was set as default and then was removed as default. Basically the frequency is only incremented if you launch Brave + it's not default

@rebron rebron changed the title [Desktop] Set browser as default dialog on 2nd launch Set browser as default dialog on 2nd launch Jan 11, 2021
@karenkliu
Copy link
Author

Check that when we implement this issue it also fixes #9306

@rebron rebron removed their assignment Jan 27, 2021
@rebron rebron removed the OS/Android Fixes related to Android browser functionality label Jan 27, 2021
@rebron rebron changed the title Set browser as default dialog on 2nd launch 'Set browser as default' dialog on 2nd and nth launch Jan 27, 2021
@simonhong
Copy link
Member

simonhong commented Jan 27, 2021

Just a note.
In chromium, dismissed infobar is re-appearred after some days passed. period_days.
In our case, fetched period_days is zero.
I don't know about this variation subsystem. Maybe we don't show again after it's dismissed first time.
@iefremov @bsclifton Do you know we configure this variation value?

  if (last_dismissed_value) {
    int period_days = 0;
    base::StringToInt(variations::GetVariationParamValue(
                          "DefaultBrowserInfobar", "RefreshPeriodDays"),
                      &period_days);
    if (period_days <= 0 || period_days == std::numeric_limits<int>::max())
      return false;  // Failed to parse a reasonable period.
    base::Time show_on_or_after =
        base::Time::FromInternalValue(last_dismissed_value) +
        base::TimeDelta::FromDays(period_days);
    if (base::Time::Now() < show_on_or_after)
      return false;
  }

@rebron @karenkliu and we don't show dialog anymore after 20th launch?

simonhong added a commit to brave/brave-core that referenced this issue Jan 29, 2021
Browser will launch window modal dialog at 2nd, 3rd, 4th, 8th and 20th
launching if browser is not a default.

If browser is changed from default to non-default, browser will launch
dialog again with same frequency.

fix brave/brave-browser#12203
simonhong added a commit to brave/brave-core that referenced this issue Feb 1, 2021
Browser will launch window modal dialog at 2nd, 3rd, 4th, 8th and 20th
launching if browser is not a default.

If browser is changed from default to non-default, browser will launch
dialog again with same frequency.

fix brave/brave-browser#12203
@simonhong simonhong added this to the 1.21.x - Nightly milestone Feb 2, 2021
simonhong added a commit to brave/brave-core that referenced this issue Feb 2, 2021
@btlechowski
Copy link

btlechowski commented Feb 16, 2021

Verification passed on

Brave 1.21.56 Chromium: 88.0.4324.152 (Official Build) dev (64-bit)
Revision 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS Ubuntu 18.04 LTS
Brave 1.21.47 Chromium: 88.0.4324.152 (Official Build) beta (64-bit)
Revision 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS Ubuntu 18.04 LTS
Brave 1.22.26 Chromium: 89.0.4389.48 (Official Build) nightly (64-bit)
Revision 0fe3c4589a6cf5ce719d167834dfa9cd8978937a-refs/branch-heads/4389@{#873}
OS Ubuntu 18.04 LTS
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 test plan from brave/brave-core#7716

Important: Make any browser except Brave (any channel) the default browser

Verified default browser dialog is shown after 2nd, 3rd, 4th, 8th and 20th launch. Verified brave->default_browser->launching_count in Preferences is increased each browser launch
image
image

Verified dialog is no longer shown after the browser is made default.

Verified dialog is shown after another browser is made default. Verified brave->default_browser->launching_count in Preferences starts from 1.

Verified dialog is not shown when brave://welcome page is active tab.

Verified on upgrade profile (1.20.x) that default browser dialog is shown.

Logged #14177 for missing translations

Encountered #13980.


Verified PASSED on nightly, dev, and beta builds, macOS 11.2.1 (Big Sur) using:

Brave 1.22.42 Chromium: 89.0.4389.58 (Official Build) nightly (x86_64)
Revision 1a139f28ecc27719439e37c6b1533cee999cb802-refs/branch-heads/4389@{#1134}
OS macOS Version 11.2.1 (Build 20D74)
Brave 1.21.66 Chromium: 88.0.4324.182 (Official Build) dev (x86_64)
Revision 73ee5087001dcef33047c4ed650471b225dd8caf-refs/branch-heads/4324@{#2202}
OS macOS Version 11.2.1 (Build 20D74)
Brave 1.21.65 Chromium: 88.0.4324.182 (Official Build) beta (x86_64)
Revision 73ee5087001dcef33047c4ed650471b225dd8caf-refs/branch-heads/4324@{#2202}
OS macOS Version 11.2.1 (Build 20D74)

using the testplan from brave/brave-core#7716

Verified:

  • both dark and light themes
  • default-browser dialog appears on the 2nd, 3rd, 4th, 8th, and 20th launches
  • the above dialog does NOT appear on launches 9-19
  • after choosing "Set as default", in the Brave dialog, then confirming with the OS-level dialog, the user is no longer prompted to set their default browser, on restart
  • after the 20th launch, regardless of choosing Not Now or Set as default the modal dialog did NOT appear

Launches:

screenshots

2nd

Screen Shot 2021-02-23 at 10 55 10 AM

3rd

Screen Shot 2021-02-23 at 10 32 48 AM

4th

Screen Shot 2021-02-23 at 10 33 44 AM

8th

Screen Shot 2021-02-23 at 10 37 21 AM

20th

Screen Shot 2021-02-23 at 10 25 58 AM

@btlechowski
Copy link

Set as QA/Blocked till we get a release channel build as the issue needs to be tested on all channels.

@LaurenWags
Copy link
Member

Removing QA/Blocked since https://github.com/brave/brave-browser/releases/tag/v1.21.68 is available

@stephendonner
Copy link

stephendonner commented Feb 25, 2021

Verified FIXED on release build, macOS 11.2.2 (Big Sur) using the testplan from brave/brave-core#7716 on:

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)

Verified:

  • both dark and light themes (only light pictured, sorry)
  • default-browser dialog appears on the 2nd, 3rd, 4th, 8th, and 20th launches
  • the above dialog does NOT appear on launches 9-19
  • after choosing "Set as default", in the Brave dialog, then confirming with the OS-level dialog, the user is no longer prompted to set their default browser, on restart
  • after the 20th launch, regardless of choosing Not Now or Set as default the modal dialog did NOT appear

Launches:

2nd 3rd 4th 8th 20th

Verification passed on

Brave | 1.21.68 Chromium: 89.0.4389.58 (Official Build) (64-bit)
-- | --
Revision | 1a139f28ecc27719439e37c6b1533cee999cb802-refs/branch-heads/4389@{#1134}
OS | Windows 10 OS Version 2004 (Build 19041.804)

  • both dark and light themes
  • default-browser dialog appears on the 2nd, 3rd, 4th, 8th, and 20th launches
  • the above dialog does NOT appear on launches 9-19
  • after choosing "Set as default", in the Brave dialog, then confirming with the OS-level dialog, the user is no longer prompted to set their default browser, on restart
  • after the 20th launch, regardless of choosing Not Now or Set as default the modal dialog did NOT appear

Launches:

2nd 3rd 4th 8th 20th
image image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment