This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 974
Add new "Auto-suggest" sites preference (payments tab) #6261
Labels
Milestone
Comments
bsclifton
changed the title
Add advanced option "exclude-by-default"
Add new "Auto-suggest" sites preference (payments tab)
Dec 19, 2016
I'd like to implement the button and the switches on the UI if it is ok |
@luixxiul appreciate your help. We'll be using |
cezaraugusto
added a commit
that referenced
this issue
Jan 6, 2017
Fix #6261 Auditors: @mrose17, @bsclifton Test Plan: as described on PR
cezaraugusto
added a commit
that referenced
this issue
Jan 8, 2017
Fix #6261 Auditors: @mrose17, @bsclifton Test Plan: as described on PR
4 tasks
Fixed with #6633 which was merged into branch Manual tests: Case 1 (Publisher exists and is disabled)
Case 2 (Publisher exists and is enabled)
Case 3 (Publisher doesn't exist yet)
Edge Case 1 (User has both "auto-suggest sites" and "only show included sites" options enabled)
Edge Case 2 (User has Payments disabled)
|
bsclifton
pushed a commit
that referenced
this issue
Jan 14, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 17, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 17, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 18, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 20, 2017
Close #6261 Auditors: @mrose17, @bsclifton
NejcZdovc
pushed a commit
to NejcZdovc/browser-laptop
that referenced
this issue
Jan 23, 2017
Close brave#6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 23, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 24, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 25, 2017
Close #6261 Auditors: @mrose17, @bsclifton
bsclifton
pushed a commit
that referenced
this issue
Jan 25, 2017
Close #6261 Auditors: @mrose17, @bsclifton
This was referenced Feb 3, 2017
This was referenced Feb 3, 2017
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Goals
Give users a toggle for auto-suggest sites
If "auto suggest sites" is disabled, let's add a little BTC icon to the right of the address bar allowing users to click to say "include this publisher for payments"
Additional info
A "permanently hide" button introduced with #6101
(this is approved but not merged- you can cherry pick the commits into the
exclude-by-default
branch in the meantime)Tasks
Default values for the two new settings
Screenshots (mockup)
same UI showing mockup for flipped switches:
New URL bar button provides a convenient "remote control" for toggling site contributions.
site contributions off: (exclude state)
site contributions on: (include state)
The text was updated successfully, but these errors were encountered: