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

Add a new setting that allows Brave Rewards icon to be hidden when inactive #2975

Closed
davidtemkin opened this issue Jan 15, 2019 · 9 comments · Fixed by brave/brave-core#1552

Comments

@davidtemkin
Copy link

davidtemkin commented Jan 15, 2019

Test plan

See brave/brave-core#1552

Description

Add a new on/off setting in the "Appearance" section, below "Use wide location bar", as follows:

"Hide Brave Rewards icon in location bar when Rewards is not enabled"

This option should default to off -- i.e., the default setting is that Brave Rewards icon is always visible, whether or not Rewards is active.

When the setting is on, the Brave Rewards icon should be hidden unless Rewards is on. If Rewards is on, the icon is visible regardless of the setting.

The setting only controls whether or not the icon is visible when Rewards is off.

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Jan 15, 2019
@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Jan 15, 2019
@eljuno
Copy link

eljuno commented Jan 16, 2019

Related to #1475?

@davidtemkin
Copy link
Author

Yes. This is how we are solving that issue.

@NejcZdovc NejcZdovc added this to the 1.x Backlog milestone Jan 16, 2019
@davidtemkin
Copy link
Author

Updated the string in initial comment.

@rebron
Copy link
Collaborator

rebron commented Jan 17, 2019

We could also do two lines:
"Show Brave Rewards button"
"Hide Brave Rewards button in address bar when Rewards is not enabled"

@rebron rebron added the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Jan 17, 2019
@davidtemkin
Copy link
Author

davidtemkin commented Jan 17, 2019

If it were two lines the second line would need to be a more detailed explanation of the first line (in keeping with Chrome style):

First line: "Always show Brave Rewards icon in location bar"
Second line: "Shows Brave Rewards icon even when Rewards is not enabled"

That could work. Don't know if it's preferable.

(BTW if we want to use "address bar" , we should change existing references to "location bar" in settings)

@rebron
Copy link
Collaborator

rebron commented Jan 17, 2019

I think the original language was right with a few updates using "button" and "address bar":

first line: Hide Brave Rewards button
second line: Hides Brave Rewards button in address bar when Rewards is not enabled

We'll change instances of location bar/url bar to address bar to be consistent.

@NejcZdovc NejcZdovc added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Jan 19, 2019
@rebron rebron assigned mkarolin and unassigned petemill and bsclifton Jan 29, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@bsclifton bsclifton removed the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Feb 7, 2019
@btlechowski
Copy link

btlechowski commented Feb 25, 2019

Verification passed on

Brave 0.61.37 Chromium: 73.0.3683.39 (Official Build) beta (64-bit)
Revision cc53b0e12fcaf42e4bab8d6c23bd4fb7aae99f6c-refs/branch-heads/3683@{#413}
OS Windows 7 Service Pack 1 Build 7601.24312

Used test plan from brave/brave-core#1552

image
image
image
image

Verification passed on

Brave 0.61.41 Chromium: 73.0.3683.39 (Official Build) beta (64-bit)
Revision cc53b0e12fcaf42e4bab8d6c23bd4fb7aae99f6c-refs/branch-heads/3683@{#413}
OS Linux mint

Verification PASSED on macOS 10.14.3 x64 using the following build:

Brave 0.61.45 Chromium: 73.0.3683.39 (Official Build) beta(64-bit)
Revision cc53b0e12fcaf42e4bab8d6c23bd4fb7aae99f6c-refs/branch-heads/3683@{#413}
OS Mac OS X

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.