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

Show "Added to sidebar" feedback bubble only three times #16018

Closed
simonhong opened this issue May 21, 2021 · 1 comment · Fixed by brave/brave-core#8911
Closed

Show "Added to sidebar" feedback bubble only three times #16018

simonhong opened this issue May 21, 2021 · 1 comment · Fixed by brave/brave-core#8911

Comments

@simonhong
Copy link
Member

simonhong commented May 21, 2021

Currently, that feedback bubble is shown everytime new items are added.
Limit three times.

@simonhong simonhong self-assigned this May 21, 2021
@simonhong simonhong added QA/Yes feature/sidebar Relating to Brave's Sidebar feature release-notes/exclude labels May 21, 2021
@simonhong simonhong changed the title Show "Added to sidebar" tooltip only three times Show "Added to sidebar" feedback bubble only three times May 24, 2021
simonhong added a commit to brave/brave-core that referenced this issue May 24, 2021
fix brave/brave-browser#16018

We will show item added feedback bubble only three times.
@simonhong simonhong added this to the 1.27.x - Nightly milestone May 24, 2021
@stephendonner
Copy link

stephendonner commented May 27, 2021

Verified PASSED using the testplan from brave/brave-core#8911 with build

Brave 1.27.25 Chromium: 91.0.4472.77 (Official Build) nightly (x86_64)
Revision 1cecd5c8a856bc2a5adda436e7b84d8d21b339b6-refs/branch-heads/4472@{#1246}
OS macOS Version 11.4 (Build 20F71)

Steps:

  1. Launched Brave with clean profile and enabled sidebar via brave://flags; relaunched
  2. Loaded brave.com
  3. Added current tab's site to sidebar
  4. Repeated steps 2 and 3 (with google.com, mozilla.com, and apple.com) and checked that the sidebar-item-added feedback bubble is displayed only three times - as I added apple.com it didn't display.
brave://flags brave.com google.com mozilla.com apple.com
Screen Shot 2021-05-27 at 4 52 32 PM Screen Shot 2021-05-27 at 4 52 38 PM Screen Shot 2021-05-27 at 4 52 48 PM Screen Shot 2021-05-27 at 4 52 57 PM Screen Shot 2021-05-27 at 4 53 10 PM

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.

2 participants