-
Notifications
You must be signed in to change notification settings - Fork 898
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
Brave VPN card widget in NTP #26189
Brave VPN card widget in NTP #26189
Conversation
e8e842b
to
407aa52
Compare
6092966
to
beef9f4
Compare
A Storybook has been deployed to preview UI for the latest push |
4160fcb
to
59a870c
Compare
Per DM with @rebron, some suggested changes:
And, I know the Rewards card copy isn't necessarily under review here, but noting the word |
a70b42a
to
6c04f4b
Compare
// BraveVPN | ||
RefreshVPNState(); | ||
LaunchVPNPanel(); | ||
OpenVPNAccountPage(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @bridiver for review
VPN card should not be rendered before receiving final purchased state.
It's not reached there if vpn card should be hidden.
67e77bc
to
5e666fa
Compare
also deleted vpn prefs handling from day zero manager as it's not used now.
5e666fa
to
0936793
Compare
[puLL-Merge] - brave/brave-core@26189 DescriptionThis PR adds a new Brave VPN widget to the New Tab Page (NTP) in Brave Browser. The widget allows users to view and control their VPN connection status directly from the NTP. ChangesChanges
Possible Issues
Security Hotspots
|
Released in v1.75.8 |
Brave VPN card widget in NTP
Brave VPN card widget in NTP
Verification PASSED using
Case 1: Initial sell screen/ NOT Subscribed state - Passed
Case 2: Customize Dashboard - Card functionality - Passed
Case 3: Sign up/Subscribe to Brave VPN via NTP widget as a New user - Passed
Case 4: Enable Brave VPN via NTP card widget in the NTP - Passed
Case 5: Default WireGuard and fallback to IKEv2 connections - Passed
Case 6 - Multiple Profiles - Passed
2024-11-30_13h39_33.mp4Case 7 - Server Selection - Passed
Case 8 - Can't connect to server UI - Passedoptimal server selection in the USA regionON via VPN card widgetCan't connect to server message is shownTry again button2024-12-02_20h04_09.mp4Case 9 -Connection consistency - Passed
2024-11-30_17h02_31.mp4Case 10 Upgrade - Passed
upgrade.mp4 |
Verification
Case 1: Initial sell screen/ NOT Subscribed state - PASSED
Case 2: Sign up/Subscribe to Brave VPN via NTP widget as a New user -PASSED
Case 3: Enable Brave VPN via NTP card widget in the NTP - PASSED
Case 4 - Server Selection - PASSED
Case 5 - Can't connect to server UI -PASSEDoptimal server selection in the USA regionON via VPN card widgetCan't connect to server message is shownTry again buttonScreen.Recording.2024-12-02.at.2.57.29.PM.movCase 6 -Connection consistency - PASSED
Screen.Recording.2024-12-02.at.5.32.29.PM.movCase 7: Customize Dashboard - Card functionality - Passed
Case 8 Upgrade - Passed
Screen.Recording.2024-12-03.at.11.19.42.AM.mov |
Verification
|
* Merge pull request #26189 from brave/brave_vpn_ntp_widget_demo Brave VPN card widget in NTP * Merge pull request #26496 from brave/follow_up_brave_vpn_card F/U vpn card widgets in NTP * Merge pull request #26689 from brave/vpn_card_group_policy Hide NTP VPN card when its group policy is disabled * Merge pull request #26743 from brave/update_vpn_card_option_image Updated vpn card option image in NTP * Merge pull request #26745 from brave/update_vpn_product_title Update VPN product title at vpn panel * Merge pull request #26746 from brave/update_trial_url_from_vpn_card Updated trial url from vpn card * Merge pull request #26765 from brave/fix_vpn_icon_size_vpn_card Fixed bottom of vpn icon of vpn card was clipped
* Merge pull request #26299 from brave/introduce_brave_vpn_controller Added BraveVPNController to BrowserWindowFeatures * Make vpn bubble start with server selection (#26310) fix brave/brave-browser#41991 NTP VPN Widget card will use this. * Merge pull request #26189 from brave/brave_vpn_ntp_widget_demo Brave VPN card widget in NTP * Merge pull request #26496 from brave/follow_up_brave_vpn_card F/U vpn card widgets in NTP * Fixed build failure * Merge pull request #26689 from brave/vpn_card_group_policy Hide NTP VPN card when its group policy is disabled * Merge pull request #26743 from brave/update_vpn_card_option_image Updated vpn card option image in NTP * Merge pull request #26745 from brave/update_vpn_product_title Update VPN product title at vpn panel * Merge pull request #26746 from brave/update_trial_url_from_vpn_card Updated trial url from vpn card * Merge pull request #26765 from brave/fix_vpn_icon_size_vpn_card Fixed bottom of vpn icon of vpn card was clipped
Resolves brave/brave-browser#41778
Screen.Recording.2024-11-04.130356.mp4
TODOs:
BraveVPNController
toBrowserWindowFeatures
brave-browser#41973Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Sign-up, primary flow, and basic functions
Free trial
button and check account page is loadedChange
link and check vpn panel is opened with server selectionCheck consistency of VPN widget with toolbar panel, hamburger menu, and taskbar
NOTE: This f/u PR(#26496) should be verified together.
Widget display for New and Existing users
New users
Existing users
Linux