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

Make Default button is not displayed in brave://settings #21954

Closed
MadhaviSeelam opened this issue Mar 29, 2022 · 3 comments
Closed

Make Default button is not displayed in brave://settings #21954

MadhaviSeelam opened this issue Mar 29, 2022 · 3 comments
Assignees
Labels
bug closed/by-author onboarding Front-end related to helping a new user learn about features within the browser. OS/Desktop OS/Windows

Comments

@MadhaviSeelam
Copy link

MadhaviSeelam commented Mar 29, 2022

Description

Make Default button is not displayed in the brave://settings when Maybe later button selected on first run dialog on Windows

Steps to Reproduce

Scenario 1: `Make Default` button should be available in `brave://settings`
  1. Clean install
  2. Launch Brave
  3. First run dialog Ready for the best privacy online? is displayed
  4. Clicked Maybe later to dismiss the dialog
  5. Navigate to brave://settings
  6. Verify Default browser entry shows Make default button

Actual result:

brave://settings displayed Brave is your default browser and Make Default button not available
However, 1.39.x - Nightly does display Make default button as expected

dialog

Expected result:

Default browser entry should show "Make default" button

Scenario 2: Click `x` on the dialog
  1. Clean Profile
  2. Launch Brave
  3. First run dialog Ready for the best privacy online? is displayed
  4. Clicked x to dismiss the dialog
  5. Navigate to brave://settings
  6. Verify Default browser entry shows Make default button

Actual result:

brave://settings displayed Brave is your default browser and Make Default button not available
However, 1.39.x - Nightly does display Make default button as expected

dialog

Expected result:

Default browser entry should show "Make default" button

Scenario 3: The dialog should display buttons same order as the mockup
  1. Clean install
  2. Launch Brave
  3. First run dialog Ready for the best privacy online? should be displayed
  4. Verify the dialog matches to the mockup

Actual result:

The order is reversed in 1.38.x and dialog show "Set Brave as default" and "Maybe later"

 dialog

Expected result:

Mockup shows "Maybe later" and "Set Brave as default" order

Reproduces how often:

Easily

Brave version (brave://version info)

Brave 1.38.70 Chromium: 100.0.4896.46 (Official Build) beta (64-bit)
Revision 5ca33821b2211805855c77d334353d27c616a7ca-refs/branch-heads/4896@{#584}
OS Windows 11 Version 21H2 (Build 22000.556)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? No

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@MadhaviSeelam MadhaviSeelam changed the title Make Default button is not displayed in the brave://settings when Maybe later button selected on first run dialog on Windows Make Default button is not displayed in the brave://settings Mar 29, 2022
@MadhaviSeelam MadhaviSeelam changed the title Make Default button is not displayed in the brave://settings Make Default button is not displayed in brave://settings Mar 29, 2022
@MadhaviSeelam MadhaviSeelam added OS/Windows onboarding Front-end related to helping a new user learn about features within the browser. bug labels Mar 29, 2022
@stephendonner
Copy link

cc @simonhong

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 12, 2022

@MadhaviSeelam Could you test Scenario 1 and 2 again? I have verified the issue, it's working for me. Please make sure brave is not set as the default browser prior to verifying this scenario. Also, if you find multiple issues, it would be better to log issues separately. Currently, I could only reproduce scenario 3 (Order of Set brave as default and Maybe later buttons is reversed). Can you log a separate issue for this? Let me know if I have missed anything.
image

@MadhaviSeelam
Copy link
Author

MadhaviSeelam commented Apr 12, 2022

Closing this as scenario 1 & scenario 2 are working as expected. Filed a new issue for scenario 3 #22255

Thanks @GeetaSarvadnya

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug closed/by-author onboarding Front-end related to helping a new user learn about features within the browser. OS/Desktop OS/Windows
Projects
None yet
Development

No branches or pull requests

5 participants