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

Rewards onboarding double overlay fix #13423

Closed
ghost opened this issue Jan 7, 2021 · 4 comments · Fixed by brave/brave-core#7556
Closed

Rewards onboarding double overlay fix #13423

ghost opened this issue Jan 7, 2021 · 4 comments · Fixed by brave/brave-core#7556

Comments

@ghost
Copy link

ghost commented Jan 7, 2021

Description

Currently if a user opens the rewards panel and dismisses the global overlay, a smaller opt-in UI is then also triggered underneath it to expose the same messaging again.

Steps to Reproduce

  1. Open a clean browser install or upgrade with Rewards off
  2. Tap the Rewards panel to see the global Rewards opt-in overlay
  3. Dismiss the overlay and you will see a second dialog box with almost the same messaging
  4. Closing that dialog will then show the full panel

Actual result:

Screen Shot 2021-01-07 at 10 34 26 AM

Expected result:

Screen Shot 2021-01-07 at 10 03 02 AM

Android Device details:

  • Install type (ARM, x86): Arm
  • Device type (Phone, Tablet, Phablet):Phone
  • Android version: Nightly 1.20.60
@ghost ghost added the OS/Android Fixes related to Android browser functionality label Jan 7, 2021
@deeppandya deeppandya self-assigned this Jan 11, 2021
@deeppandya deeppandya added this to the 1.20.x - Nightly milestone Jan 11, 2021
@deeppandya
Copy link

@anthonypkeane @bsclifton where should we land this ? it's merged to 1.20 for now.

@bsclifton
Copy link
Member

@deeppandya I think 1.20 works - we're pretty close to verifying 1.19
cc: @kjozwiak @LaurenWags just in case we did want this in 1.19

@LaurenWags
Copy link
Member

1.20.x sounds good to me 👍🏻

@srirambv
Copy link
Contributor

Verification passed on OnePlus 6T with Android 10 running 1.20.85 x64 beta build

  • Verified second onboarding overlay is not shown when the global onboarding is dismissed
ARM-Onboarding.mp4

Verification passed on Samsung Tab A with Android 10 running 1.20.85 x64 beta build

  • Verified second onboarding overlay is not shown when the global onboarding is dismissed
Tab.Onboarding.mp4

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.

5 participants