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

Different private profile button style #8807

Closed
simonhong opened this issue Mar 23, 2020 · 8 comments · Fixed by brave/brave-core#6912
Closed

Different private profile button style #8807

simonhong opened this issue Mar 23, 2020 · 8 comments · Fixed by brave/brave-core#6912
Assignees

Comments

@simonhong
Copy link
Member

From current master build(Version 1.8.31 Chromium: 81.0.4044.69 (Developer Build) (64-bit)):
image

From current beta build(Version 1.7.67 Chromium: 80.0.3987.149 (Official Build) beta (64-bit)):
image

@rebron
Copy link
Collaborator

rebron commented Mar 23, 2020

cc: @tomlowenthal
Button selected state is wrong but placement of [icon-text] vs [text-icon] looks better?

@rebron
Copy link
Collaborator

rebron commented Mar 23, 2020

cc: @karenkliu can you take a look? This is on the latest nightly https://github.com/brave/brave-browser/releases/tag/v1.8.32

@tildelowengrimm
Copy link
Contributor

No opinion on the order of the elements. But this is the first time I learned that this was a button rather than a badge. As far as I know, this was never specified as a button, and I’d prefer if it went back to being a badge.

@karenkliu
Copy link

☝️ what @tomlowenthal said.

This order [icon-text] is correct:

Screen Shot 2020-03-24 at 3 33 40 PM

But it should have the same background color as the toolbar and should not be clickable.

@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. OS/Desktop labels May 8, 2020
@rebron
Copy link
Collaborator

rebron commented May 8, 2020

cc: @mkarolin @petemill

@mkarolin
Copy link
Contributor

mkarolin commented May 8, 2020

The additional functionality from Chromium here is for when multiple Private windows were open. Then you have the window count in the button and the menu option to exit would exit them all at once. Not sure if we care for such functionality.
image

@bsclifton
Copy link
Member

This is a nice feature; what action should we take with this ticket? Is the problem that it looks like a button?

@karenkliu
Copy link

karenkliu commented Jun 2, 2020

@mkarolin I see - thanks for pointing this out.

@bsclifton I suspect that the "Private" label became clickable after Chrome added some additional functionality and we inherited it. For private windows the issue is styling. This is what I recommend:

Style the "Private" button like a Brave button so users know there is functionality there
private window 1

On hover, show a hover state:
private window 2 -hover

On click, show a panel tweaked so the Private icon doesn't look like another button and with Brave styling + icons:
private window 3 - panel

When multiple windows are open, show this text treatment:
private window 4 - multiple windows

We should support dark theme for the panel:
dark theme panel

TOR windows also inherits this functionality from Chrome but it doesn't really work:

  • TOR panel doesn't show the number of TOR windows open
  • Clicking "Exit Tor" doesn't close the window(s)

Screen Shot 2020-06-01 at 5 01 28 PM

Should we file a separate issue to fix this in Tor? I would recommend doing the same treatment as our Private windows to keep interactions consistent; also keeping the sunglasses icon for Tor windows:
tor window 1
tor window 2 -hover
tor window 3 - panel
tor window 4 - multiple windows

Figma link for reference: https://www.figma.com/file/5THkuEtO2Ewn9LfqrHZP9a/Desktop-Private-Tabs-and-TOR?node-id=4%3A4737

@srirambv srirambv changed the title Different private profile button style [Desktop] Different private profile button style Sep 9, 2020
@simonhong simonhong self-assigned this Dec 8, 2020
@simonhong simonhong added this to the 1.19.x - Beta milestone Dec 8, 2020
@rebron rebron changed the title [Desktop] Different private profile button style Different private profile button style Jan 4, 2021
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.

6 participants