Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Auto include button is too small #7087

Closed
luixxiul opened this issue Feb 6, 2017 · 5 comments
Closed

Auto include button is too small #7087

luixxiul opened this issue Feb 6, 2017 · 5 comments

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Feb 6, 2017

Test plan:

#7122 (comment)


Describe the issue you encountered: Auto include button is too small. This is the cropped picture which I took with my smartphone.

2017-02-06 16 24 48

The verified mark is less than 1 millimeter I think :-o

  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 10

  • Brave Version (revision SHA): 0.13.2 RC6

  • Steps to reproduce:

    1. Open about:preferences#payments
    2. Disable "auto-include"
    3. Go to https://brianbondy.com
  • Actual result: The icon is so small and not clear what it means

  • Expected result: The icon, including the verification mark, should be enough large to make it clear what it means.

  • Any related issues: Refinements to Add Funds URL Include Button #6754

@srirambv
Copy link
Collaborator

srirambv commented Feb 6, 2017

Can make the entire container green for verified publishers
image

cc: @bradleyrichter

@luixxiul
Copy link
Contributor Author

luixxiul commented Feb 6, 2017

Otherwise how about making the verified icon so large as the badge: #7050 ?

@srirambv
Copy link
Collaborator

srirambv commented Feb 6, 2017

I think the badge is roughly 3/4th the size of icon itself which would cover most of it.

@bradleyrichter
Copy link
Contributor

The amount of green needs to be small enough to not confuse the user into thinking they are visiting a safe site and accidentally enter their sensitive information.

The verified information is secondary and not nearly as important as whether the site is marked for funding.

I'll try enlarging the tiny verified badge so it is just "very small" instead of "tiny".

@bradleyrichter
Copy link
Contributor

new SVGs for this issue:

fund_icons.zip

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.