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

Brave Talk onboarding modal persists on all new tabs unless dismissed via close button #13492

Closed
srirambv opened this issue Jan 12, 2021 · 4 comments · Fixed by brave/brave-core#9616
Assignees
Labels
enhancement feature/new-tab OS/Desktop polish Nice to have — usually related to front-end/visual tasks premium All issues related to Brave Premium priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Win64 QA/Yes release-notes/exclude

Comments

@srirambv
Copy link
Contributor

Description

With new onboarding for Brave Together, the onboarding modal is only dismissed when you click on the X button. Even if you click on the together shortcut and start a video call and then open a second new tab Brave Together onboarding is still shown

Steps to Reproduce

  1. Update to 1.20.73 to get Brave together onboarding on an NTP
  2. Click on the together button to start a video call
  3. Open a new tab, still keeps showing Brave Together onboarding and covers Rewards widget

Actual result:

image

Expected result:

Should dismiss onboarding if user clicks and starts a video call or navigates to a new tab

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.20.73 Chromium: 88.0.4324.79 (Official Build) nightly (64-bit)
Revision bd1e9353659b2491dac971226a973ca3b5684a14-refs/branch-heads/4324@{#1520}
OS All

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? NA
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

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

Miscellaneous Information:

cc: @petemill @bsclifton

@srirambv srirambv added enhancement feature/new-tab polish Nice to have — usually related to front-end/visual tasks QA/Yes OS/Desktop labels Jan 12, 2021
@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Apr 19, 2021
@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P4 Planned work. We expect to get to it "soon". labels Jul 28, 2021
@rebron rebron changed the title Brave Together onboarding modal persists on all new tabs unless dismissed via close button Brave Talk onboarding modal persists on all new tabs unless dismissed via close button Jul 28, 2021
@karenkliu
Copy link

Thank you for filing this!

@kjozwiak
Copy link
Member

kjozwiak commented Aug 9, 2021

Adding QA/Blocked until we get Brave Talk enabled on main (staging) via Griffin. After it's enabled on staging, QA will be able to run through the above verification. Once we get closer to Brave Talk release date, we'll push the same change into production.

@kjozwiak
Copy link
Member

kjozwiak commented Aug 9, 2021

Removing QA/Blocked as brave/brave-variations#93 was merged into main (Griffin staging).

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Aug 10, 2021

Verification passed on

Brave | 1.28.105 Chromium: 92.0.4515.131 (Official Build) (64-bit)
-- | --
Revision | 6b8d6c56ce21e38a72f7c4becb5abc1fa5134f29-refs/branch-heads/4515@{#1933}
OS | Windows 10 OS Version 2009 (Build 19043.1110)

  • ensured that Brave Talk widget was enabled via NTP on main (staging) after launching brave via the following:
    brave.exe --variations-server-url=https://variations.bravesoftware.com/seed

  • ensured that BraveTalkStudy is enabled via brave://version/?show-variations-cmd
    image

  • Verified the test plan from New Tab Page: Brave Talk prompt show and hide spec brave-core#9616

  • ensured that the BraveTalk promo tooltip is shown in NTP when the OS clock is forwarded to 3 days

now after 3 days
image image
  • Confirmed that the Brave Talk promo tooltip is not hidden in the NTP when NTP with promo tooltip is active for more than 4 secs
  • Confirmed that the Brave Talk promo tooltip is hidden in NTP when NTP is refreshed/reload
  • Confirmed that the Brave Talk promo tooltip is hidden when a new NTP is opened
initial display after 4+ seconds and refresh new-tab page
image image image
  • Confirmed Brave Talk opens up on https://talk.brave.com/widget when Try it out is clicked in an NTP promo tooltip
  • Confirmed Brave Talk promo tooltip is hidden in a new NTP
Brave Talk promo shown opens talk.brave.com new-tab page
image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature/new-tab OS/Desktop polish Nice to have — usually related to front-end/visual tasks premium All issues related to Brave Premium priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Win64 QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants