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

Add Keep in dock option to first run dialog and settings #24055

Closed
rebron opened this issue Jul 14, 2022 · 7 comments · Fixed by brave/brave-core#14714
Closed

Add Keep in dock option to first run dialog and settings #24055

rebron opened this issue Jul 14, 2022 · 7 comments · Fixed by brave/brave-core#14714
Assignees
Labels
design A design change, especially one which needs input from the design team OS/Desktop OS/macOS priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-macOS QA/Yes release-notes/include

Comments

@rebron
Copy link
Collaborator

rebron commented Jul 14, 2022

Description

When user sets Brave as a default browser, "Keep Brave in dock" also.

Steps to Reproduce

  1. Launch Brave with clean profile
  2. Click Set Brave as default button after checking Keep in dock
  3. Check Brave is added to dock
  4. Load brave://settings/getStarted and check settings shows as Brave is already in dock
  5. Unpin from taskbar and reload brave://settings/getStarted and click Dock button
  6. Check Brave is added to dock

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? n/a

Other Additional Information:

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

Miscellaneous Information:

See related: #24054

@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. OS/macOS QA/Yes release-notes/include labels Jul 14, 2022
@rebron rebron changed the title Keep in dock when user set Brave as a default browser Keep in dock when user sets Brave as a default browser Jul 14, 2022
@sangwoo108
Copy link

I guess chrome/browser/mac/dock.h can be used for this

@bridiver
Copy link
Contributor

after speaking to @rebron we should add a checkbox if you want to keep it in the dock (unchecked by default)

@bridiver
Copy link
Contributor

same dialog ^^

@rebron rebron added design A design change, especially one which needs input from the design team design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. labels Aug 22, 2022
simonhong added a commit to brave/brave-core that referenced this issue Aug 23, 2022
@aguscruiz
Copy link

@aguscruiz aguscruiz removed the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Sep 21, 2022
@simonhong simonhong changed the title Keep in dock when user sets Brave as a default browser Add Keep in dock option to first run dialog and settings Oct 26, 2022
simonhong added a commit to brave/brave-core that referenced this issue Oct 27, 2022
fix brave/brave-browser#24055

User can add to dock via first run or settings.
@brave-builds brave-builds added this to the 1.47.x - Nightly milestone Oct 28, 2022
@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Nov 3, 2022
@stephendonner
Copy link

Design here https://www.figma.com/file/tLXWGCpNoiJxDZDdpfordj/Desktop-Settings?node-id=2952%3A45315

image

The Figma assets are pretty clearly -- though labeled for macOS -- a Windows screenshot with a bit of macOS treatment.

@aguscruiz can you please confirm that brave/brave-core#14714 (comment) is what's intended to be implemented, for macOS? I'm 99% sure but want to be 110%; thanks.

cc: @LaurenWags

@stephendonner stephendonner removed the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Nov 4, 2022
@kjozwiak
Copy link
Member

kjozwiak commented Nov 4, 2022

The above requires 1.46.94 or higher for 1.46.x verification 👍

@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Nov 7, 2022
@stephendonner
Copy link

stephendonner commented Nov 7, 2022

Verification PASSED using

Brave 1.46.96 Chromium: 107.0.5304.91 (Official Build) beta (x86_64)
Revision 3d5948960d62418160796d5831a4d2d7d6c90fa8-refs/branch-heads/5304@{#1097}
OS macOS Version 11.7.1 (Build 20G918)

Prerequisite: make a non-Brave browser the system default via Apple menu -> System Preferences... -> General -> Default web browser

Screen Shot 2022-11-03 at 1 52 02 PM

Case 1: Pin Brave to dock via the Pin to Dock checkbox in the first-run dialog - PASSED

  1. installed 1.46.96
  2. launched Brave
  3. checked the Keep in Dock checkbox
  4. clicked on the Set Brave as default button
  5. confirmed I got the Do you want to change your default web browser to "Brave" or keep using "Firefox"? dialog
  6. clicked on Use "Brave"
  7. confirmed Brave is kept in the dock
  8. closed Brave
  9. relaunched by clicking on the Brave icon in the dock
  10. opened brave://settings/getStarted
  11. confirmed Brave is your default browser text
  12. confirmed Brave is already in Dock text

Confirmed Brave is successfully kept in the dock

step 3 step 5 step 7 steps 11-12
Screen Shot 2022-11-07 at 10 58 13 AM Screen Shot 2022-11-07 at 10 58 21 AM Screen Shot 2022-11-07 at 10 58 48 AM Screen Shot 2022-11-07 at 10 59 17 AM

Case 2: Unpin Brave from the dock and Pin to Dock it from brave://settings/getStarted - PASSED

  1. continued from Case 1, with Brave kept in the dock
  2. context-clicked on Brave and chose Options -> Keep in Dock
  3. reloaded brave://settings/getStarted
  4. confirmed the Dock button beside Keep in Dock is clickable
  5. clicked Dock
  6. confirmed I now saw Brave is already in Dock
  7. context-click on Brave

Confirmed Brave is docked and shows Options -> Keep in Dock enabled

step 4 step 6 step 7
Screen Shot 2022-11-07 at 11 03 18 AM Screen Shot 2022-11-07 at 11 03 32 AM Screen Shot 2022-11-07 at 11 03 45 AM

Case 3: With Pin to Dock unchecked (default), click Set Brave as default on the first-run dialog - PASSED

  1. installed 1.46.96
  2. launched Brave
  3. left Keep in Dock unchecked (default)
  4. click Set Brave as default button
  5. clicked Use "Brave" on the next dialog
  6. clicked Skip welcome tour
  7. opened brave://settings/getStarted
  8. confirmed Brave is your default browser text is shown
  9. confirmed Dock button is shown in next to Keep in Dock
  10. clicked on the Dock button
  11. confirmed the Dock button on brave://settings/getStarted disappears
  12. context-click on Brave

Confirmed Brave is docked and shows Options -> Keep in Dock enabled

step 4 step 5 step 8-9 step 11 step 12
Screen Shot 2022-11-07 at 11 22 44 AM Screen Shot 2022-11-07 at 11 22 50 AM Screen Shot 2022-11-07 at 11 23 25 AM Screen Shot 2022-11-07 at 11 23 37 AM Screen Shot 2022-11-07 at 11 23 56 AM

Case 4: Pin to Dock Brave via brave://settings/getStarted when a non-Brave browser is set as default via Apple menu' -> System Preferences -> General Default web browser - PASSED

  1. installed 1.46.96
  2. launched Brave
  3. clicked Maybe later
  4. clicked Skip welcome tour
  5. closed Brave
  6. relaunched Brave
  7. clicked on Set as default in the infobar
  8. clicked on Use "Brave"
  9. opened brave://settings/getStarted
  10. confirmed it read Brave is your default browser
  11. confirmed the Dock button is enabled
  12. clicked on the Dock button
  13. confirmed the Dock button disappeared
  14. confirmed it read Brave is already in Dock
  15. context-clicked Brave

Confirmed Brave is docked and shows Options -> Keep in Dock enabled

step 3 step 7 step 8 steps 10-11 steps 13-14 step 15
Screen Shot 2022-11-07 at 11 30 10 AM Screen Shot 2022-11-07 at 11 32 30 AM Screen Shot 2022-11-07 at 11 32 41 AM Screen Shot 2022-11-07 at 11 33 00 AM Screen Shot 2022-11-07 at 11 33 09 AM Screen Shot 2022-11-07 at 11 33 25 AM

Case 5: Upgrade from 1.45.100 (beta) -> 1.46.96 (beta) - PASSED

  1. installed 1.45.100
  2. launched Brave
  3. clicked Set Brave as default on the first-run dialog
  4. clicked Use "Brave"
  5. opened brave://settings/getStarted and confirmed it shows Brave is your default browser text
  6. upgraded to 1.46.96
  7. opened brave://settings/getStarted
  8. clicked the Dock button
  9. confirmed the Dock button disappeared, and the text now reads "Brave is already in Dock`
  10. context-clicked Brave

Confirmed Brave is docked and shows Options -> Keep in Dock enabled

step 3 step 4 step 5 step 8 step 9 step 10
Screen Shot 2022-11-07 at 11 43 56 AM Screen Shot 2022-11-07 at 11 44 05 AM Screen Shot 2022-11-07 at 12 27 42 PM Screen Shot 2022-11-07 at 11 45 46 AM Screen Shot 2022-11-07 at 11 45 53 AM Screen Shot 2022-11-07 at 11 46 04 AM

@stephendonner stephendonner added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design A design change, especially one which needs input from the design team OS/Desktop OS/macOS priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-macOS QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants