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

Little indication of IPFS tab selected in settings #16111

Closed
0x1a8510f2 opened this issue May 27, 2021 · 7 comments · Fixed by brave/brave-core#9301
Closed

Little indication of IPFS tab selected in settings #16111

0x1a8510f2 opened this issue May 27, 2021 · 7 comments · Fixed by brave/brave-core#9301
Assignees
Labels
design A design change, especially one which needs input from the design team feature/web3/ipfs OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-macOS QA/Yes release-notes/exclude

Comments

@0x1a8510f2
Copy link

0x1a8510f2 commented May 27, 2021

Description

All settings tabs have icons which change when the tab is selected, except for IPFS which stays grey regardless of selection status.

Steps to Reproduce

  1. Open settings
  2. Select any tab other than IPFS
  3. Notice that the icon turns orange
  4. Select IPFS tab
  5. Notice that the icon does not turn orange

Actual result:

IPFS icon shows no indication of selected tab

Expected result:

IPFS icon changes colour to indicate selected tab

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.25.68 Chromium: 91.0.4472.77 (Official Build) (64-bit)
Revision 1cecd5c8a856bc2a5adda436e7b84d8d21b339b6-refs/branch-heads/4472@{#1246}
OS Linux

Version/Channel Information:

  • Can you reproduce this issue with the current release? YES
  • Can you reproduce this issue with the beta channel? UNTESTED
  • Can you reproduce this issue with the nightly channel? UNTESTED

Other Additional Information:

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

Screenshots:

Wallet Extension Getting started IPFS
image image image image

On a somewhat-related note, I believe the hightlighted Wallet tab icon could be improved - it's much less highlighted than the others which makes it stick out IMO.

@rebron rebron added the design A design change, especially one which needs input from the design team label May 28, 2021
@rebron
Copy link
Collaborator

rebron commented May 28, 2021

@karenkliu Do we have the right states for IPFS mocked up for selection?

@rebron
Copy link
Collaborator

rebron commented May 28, 2021

cc: @spylogsster

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label May 28, 2021
@karenkliu
Copy link

@rebron I'm waiting for the settings including side nav states to be fixed here: #13688

It's supposed to look like this:
image

IPFS should have the gradient color and indicator when it's selected.

@bbondy
Copy link
Member

bbondy commented Jun 23, 2021

@karenkliu can you point to the figma asset to use that fixes this?

@stephendonner
Copy link

Verified PASSED using

Brave 1.28.54 Chromium: 92.0.4515.81 (Official Build) nightly (x86_64)
Revision dba3af0a9a48954c8b188bc641e651580b7cb7b5-refs/branch-heads/4515@{#1162}
OS macOS Version 11.4 (Build 20F71)
selected selected hover hover
Screen Shot 2021-07-06 at 3 51 55 PM Screen Shot 2021-07-06 at 3 54 36 PM Screen Shot 2021-07-06 at 3 52 52 PM Screen Shot 2021-07-06 at 3 53 04 PM

@karenkliu
Copy link

karenkliu commented Jul 6, 2021

@stephendonner Dark theme hover seems to be using the incorrect color, and therefore not passing accessibility standards. It should be this:
Screen Shot 2021-07-06 at 4 35 47 PM
737ADE

Also, looks like the icon in the sidebar is missing the selected and hover color, as well as the selected accent line:
Screen Shot 2021-07-06 at 4 39 07 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design A design change, especially one which needs input from the design team feature/web3/ipfs OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-macOS QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants