-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 Pin to taskbar option
to first run dialog and settings
#24054
Comments
fix brave/brave-browser#24054 * Pin to taskbar from first run or default browser dialog when user want to do it. * Pin to taskbar from settings. Use upstream's api for pin state checking pin to taskbar With this, this pin to shortcut feature is available also on Win7/8.
@GeetaSarvadnya could you please give this one a check on Win 10 as well? once you're done you can add the |
Verification PASSED on
Case 1: Pin Brave to taskbar via
Confirmed Brave is successfully pinned to taskbar
Case 2: Unpin Brave from the taskbar and
Confirmed Brave successfully pinned to taskbar
Case 3: Do not check
Confirmed Brave successfully pinned to taskbar
Case 4:
Confirmed Brave successfully pinned to taskbar
|
step 1 | step 3 | step 5 | step 6-8 |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Case 2: Install 1.45.x, click on may be later then upgrade the profile 1.46.x_PASSED
Pre-requisite: same channels for the upgrade (i.e. beta-->beta)
- Install 1.45.104
- launch Brave
- Click
May be later
on the first run dialog - Open brave://settings/help and then upgrade the profile
- Upgrade to 1.46.x
- Go to brave://settings/getStarted and confirmed Pin text is shown
- Click
Pin
button - Confirmed brave is pinned to task bar and text "Brave is already pinned" is shown in the brave://settings/getStarted
Case 3: Install 1.45.x, click on `x` in the first run dialog and pin to taskbar then upgrade the profile 1.46.x_PASSED
Pre-requisite: same channels for the upgrade (i.e. beta-->beta)
- Install 1.45.104
- launch Brave
- Click
x
on the first run dialog - Pin to taskbar by using the taskbar context menu
- Upgrade to 1.46.81
- Go to brave://settings/getStarted
- Ensured
Brave is already pinned
text is shown and right click on the taskbar menu shows theUnpin from taskbar
menu option as expected - Unpin from the taskbar
- Reload brave://settings/getStarted
- Pin option is shown, click on it
- Confirmed brave is pinned and same is shown in the taskbar as well as in brave://settings/getStarted
Example | Example | Example | Example | Example | Example |
---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Pin to taskbar option
to first run dialog and settings
Description
When user set Brave as a default browser, pin Brave to taskbar also.
Steps to reproduce
Set Default
button after checkingpin to taskbar
pinned
Pin
buttonBrave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
See related: #24055
FF - https://github.com/mozilla/gecko-dev/blob/master/browser/components/shell/nsWindowsShellService.cpp
The text was updated successfully, but these errors were encountered: