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

[Desktop] Update Brave Rewards panel within welcome UI #13119

Closed
zenparsing opened this issue Dec 9, 2020 · 4 comments · Fixed by brave/brave-core#7467
Closed

[Desktop] Update Brave Rewards panel within welcome UI #13119

zenparsing opened this issue Dec 9, 2020 · 4 comments · Fixed by brave/brave-core#7467

Comments

@zenparsing
Copy link

The welcome UI should be updated with new designs for the Brave Rewards onboarding panel.

Screen Shot 2020-12-09 at 1 11 27 PM

When the user clicks the "Start Using Rewards" button, a new tab should open for the rewards settings page with ads and AC enabled.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jan 7, 2021

Verification passed on


Brave | 1.19.76 Chromium: 87.0.4280.101 (Official Build) dev (64-bit)
-- | --
Revision | 9407c80213cda69c2b7abcb4fa8e3f74488f4956-refs/branch-heads/4280@{#1807}
OS | Windows 10 OS Version 2004 (Build 19041.685)

Clean profile

Upgrade profile

  • Install 1.17.x dev build and upgrade to 1.19.x and navigate to Brave rewards in brave://welcome screen and ensured that rewards slide content and graphics is updated as the per the design mentioned in the description, click on Start using Rewards opens up brave://rewards/ page in a new tab with Ads and Auto-Contribute switches enabled
  • Install 1.17.x dev build and enable rewards via BR panel or NTP, upgrade to 1.19.x and navigate to Brave rewards in brave://welcome screen, click on Start using Rewards button redirects to brave://rewards tab which is already opened in 1.17.x

Verification passed on

Brave 1.19.78 Chromium: 87.0.4280.101 (Official Build) dev (64-bit)
Revision 9407c80213cda69c2b7abcb4fa8e3f74488f4956-refs/branch-heads/4280@{#1807}
OS Ubuntu 18.04 LTS

Clean profile

Upgrade profile

  • Install 1.17.x build and upgrade to 1.19.x and navigate to Brave rewards in brave://welcome screen and ensured that rewards slide content and graphics is updated as the per the design mentioned in the description, click on Start using Rewards opens up brave://rewards/ page in a new tab with Ads and Auto-Contribute switches enabled

Verified passed with

Brave | 1.19.82 Chromium: 88.0.4324.79 (Official Build) dev (x86_64)
-- | --
Revision | bd1e9353659b2491dac971226a973ca3b5684a14-refs/branch-heads/4324@{#1520}
OS | macOS Version 10.15.7 (Build 19H15)

Verified test plan from brave/brave-core#7467 (comment)
Verified items listed in brave/brave-core#7467 (comment)

Confirmed rewards slide of onboarding for both light and dark themes
Light Dark
Light Dark
Confirmed Terms of Service link is shown. When clicked a new tab is opened which navigates to `https://basicattentiontoken.org/user-terms-of-service/` as expected.
Light Dark
Light-TOS Dark-TOS
Confirmed Privacy Policy link is shown. When clicked a new tab is opened which navigates to `https://brave.com/privacy/browser/#rewards` as expected.
Light Dark
Light-PP Dark-PP

Confirmed light and dark theme graphic designs from figma: https://www.figma.com/file/0dbMYs8Wp8W3RgrG1yTdki/RE-1.0?node-id=99%3A0

Confirmed when clicking on "Start using Rewards" button a new tab opens to show brave://rewards with Ads and AC enabled. Start using Rewards

Per brave/brave-core#7467 (comment), confirmed that there is no onboarding tour option displayed in 1.19.x when selecting "Start using Rewards" from brave://welcome.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jan 7, 2021

@zenparsing In a clean profile 1.19.x or upgraded profile, If I enable rewards via NTP or BR panel and then turn OFF the Ads and Auto-Contribute switches in brave://rewards tab and then enable rewards via brave://welcome screen by clicking on Start using rewards , a new tab brave://rewards is opened up with Ads and Auto-Contribute switches enabled, shouldn't it point to already opened brave://rewards tab?

@zenparsing
Copy link
Author

@GeetaSarvadnya After we introduced rewards everywhere in 1.18 and removed the global rewards opt-in, the "Start using Rewards" buttons now just enable ads and AC. I think this is expected behavior, but cc @codybanxs for UX perspective.

@ghost
Copy link

ghost commented Jan 7, 2021

If I am understanding this correctly @GeetaSarvadnya, the 2nd time you click "Start using Rewards" it should open up the rewards settings page with those settings turned on. Now I believe the logic that is baked in here is that it will automatically open the rewards settings page once you opt-in regardless of what other tabs are open prior to that interaction. I am not sure if we have a way to tell if the settings page is open or not and to just use the existing screen @zenparsing.

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

Successfully merging a pull request may close this issue.

6 participants