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

Improve ability to recognize the difference between a private window and private window with Tor #2735

Closed
Qix- opened this issue Dec 25, 2018 · 11 comments · Fixed by brave/brave-core#6912
Assignees
Labels
design A design change, especially one which needs input from the design team feature/private-browsing feature/tor OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include

Comments

@Qix-
Copy link

Qix- commented Dec 25, 2018

Description

Brave version: Version 0.58.18 Chromium: 71.0.3578.98 (Official Build) (64-bit)

In the new flashy Brave UI, it's almost impossible to tell if the tab is private or private with Tor. This is a severe downgrade from the old UI where it was at least easily recognizable at a glance.

This, along with #2734, make for a pretty sizable setback in Tor experience in Brave :/

Design

  • Differentiate Private Window from Private Window with Tor by changing tab and toolbar colors
  • The color changes would also increase contrast between active and inactive tabs by darkening tab background and lightening active tab+toolbar
  • Decrease contrast of tab separator: private window #3F3256, Private Window with Tor #5A5366
  • Differentiate Private Window with a "Private Window" badge on the right edge of toolbar
  • Differentiate Private Window with Tor with a "Private with Tor" badge on the right edge of toolbar and custom graphic on tab bar background

image

Contrast ratio improves:

image

image

Contrast ratio improves:
image

image

Contrast ratio improves:

image

Assets

Figma: https://www.figma.com/file/sS3l8tqUFxt54MEiBTalUI/Desktop-toolbar-and-menu?node-id=117%3A11091

Notes

Follow up issue to #8576

@srirambv srirambv added feature/tor design A design change, especially one which needs input from the design team feature/private-browsing labels Dec 25, 2018
@srirambv srirambv added this to the 1.x Backlog milestone Dec 25, 2018
@srirambv
Copy link
Contributor

cc: @tomlowenthal @brave/design

@srirambv
Copy link
Contributor

TBH on Linux with GTX dark theme by default its only possible to distuingush if you have a new tab open which makes it clear what kind of window you are on
image

On Windows, you'd have different taskbar icons for each Private/Tor/Normal windows so its easy to guess which window is which but on Linux all the windows gets clubbed under one icon and keyboar shortcut isn't that user friendly to switch tabs of the same browser profile

screenshot from 2018-12-28 10-54-10

Unless we can override the chrome colour profile on the Guest Window I don't think it will be much helpful in terms of UI distinction of Private/Tor windows

cc: @petemill @simonhong

@Qix-
Copy link
Author

Qix- commented Dec 28, 2018

Well in the old version you could see "tor" in the tab. That was helpful enough. You'd also see "connecting to tor" in the address bar.

However, I don't see any of that anymore which leads me to believe that I'm not actually connected, which gives me lots of anxiety (: especially since things feel faster to connect in the new brave, which generally isn't a good sign when dealing with Tor.

So now I'm left with doubt about whether or not Brave is working as intended. The cognitive stress is much, much higher.

@tildelowengrimm
Copy link
Contributor

In private windows with Tor, the word "Tor" is visible on the right, next to the menu button. On regular private windows, you see a pair of sunglasses there instead. But that's certainly not an easy way to tell windows apart at a glance, from menu bars, or so on.

@tildelowengrimm tildelowengrimm added the priority/P4 Planned work. We expect to get to it "soon". label Jan 24, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@rossmoody rossmoody self-assigned this May 31, 2019
@rossmoody rossmoody removed their assignment Jul 4, 2019
@NumDeP
Copy link

NumDeP commented Sep 12, 2019

Please keep in mind Session-Tabs when working on this feature because I remember this was also a concern for many users for when dozens of tabs started to shrink different tab types in the Muon version of Brave.

@rebron rebron changed the title Extremely hard to tell if tab is private or Tor in new version [Desktop] Extremely hard to tell if tab is private or Tor in new version Sep 29, 2020
@rebron rebron assigned simonhong and unassigned karenkliu Sep 30, 2020
@simonhong simonhong added this to the 1.18.x - Nightly milestone Oct 21, 2020
@bbondy
Copy link
Member

bbondy commented Oct 30, 2020

@simonhong is this issue release blocking? If so could you please add a label? If not please remove it from the milestone.
Screen Shot 2020-10-30 at 1 22 17 PM
https://github.com/brave/brave-browser/wiki/Triage-Guidelines#milestones-and-issues

@bbondy bbondy removed this from the 1.18.x - Nightly milestone Oct 30, 2020
@simonhong
Copy link
Member

simonhong commented Nov 3, 2020

@karenkliu Can you share each theme(dark/private/tor)'s new hover color of location bar/omnibox popup also?
Or location/omnibox color change is not this issues range? (There color seems also changed in Figma)

@karenkliu
Copy link

karenkliu commented Nov 3, 2020

@simonhong

Location bar hover border color

  • Dark Theme: no change
  • Private Window: #C6B3FF 2px border
  • Private Window with Tor: #CFABE2 2px border

Omnibox background color

  • Dark Theme: #181A21
  • Private Window: #0B0724
  • Private Window with Tor: #0B0724

Figma: https://www.figma.com/file/sS3l8tqUFxt54MEiBTalUI/?node-id=117%3A11090

@LaurenWags
Copy link
Member

Questions sent to @rebron for clarification, marking as QA/Blocked for now

@rebron rebron changed the title [Desktop] Extremely hard to tell if tab is private or Tor in new version Improve ability to recognize the difference between a private window and private window with Tor Jan 4, 2021
@karenkliu
Copy link

I verified the designs on Windows and noted no color discrepancies between platforms.

Tested:

  • MacOS Version 1.20.68 Chromium: 88.0.4324.51 (Official Build) nightly (x86_64)
  • Windows Version 1.20.68 Chromium: 88.0.4324.51 (Official Build) nightly (64-bit)

@LaurenWags
Copy link
Member

LaurenWags commented Jan 8, 2021

Verified passed with

Brave | 1.19.79 Chromium: 87.0.4280.141 (Official Build) dev (x86_64)
-- | --
Revision | 9f05d1d9ee7483a73e9fe91ddcb8274ebcec9d7f-refs/branch-heads/4280@{#2007}
OS | macOS Version 10.15.7 (Build 19H15)

Confirmed visual differences between dark theme normal windows, private windows, and Tor windows.
Confirmed text on profile bubbles for Private and Tor windows as per brave/brave-core#6912 (comment).

Dark theme example Dark theme example
Screen Shot 2021-01-08 at 4 37 28 PM Screen Shot 2021-01-08 at 4 37 17 PM
Private window example Private window example Private window example
Screen Shot 2021-01-08 at 4 37 47 PM Screen Shot 2021-01-08 at 4 37 56 PM Screen Shot 2021-01-08 at 4 38 09 PM
Tor window example Tor window example Tor window example
Screen Shot 2021-01-08 at 4 38 32 PM Screen Shot 2021-01-08 at 4 38 40 PM Screen Shot 2021-01-08 at 4 38 47 PM

Verification passed on

Brave 1.19.81 Chromium: 87.0.4280.141 (Official Build) dev (64-bit)
Revision 9f05d1d9ee7483a73e9fe91ddcb8274ebcec9d7f-refs/branch-heads/4280@{#2007}
OS Ubuntu 18.04 LTS

Confirmed visual differences between dark theme normal windows, private windows, and Tor windows.
Confirmed text on profile bubbles for Private and Tor windows as per brave/brave-core#6912 (comment).

Dark theme example Dark theme example
image image
Private window example Private window example Private window example
image image image
Tor window example Tor window example Tor window example
image image image

Verification passed on


Brave | 1.19.83 Chromium: 88.0.4324.79 (Official Build) (64-bit)
-- | --
Revision | bd1e9353659b2491dac971226a973ca3b5684a14-refs/branch-heads/4324@{#1520}
OS | Windows 10 OS Version 2004 (Build 19041.685)

Confirmed visual differences between dark theme normal windows, private windows, and Tor windows.
Confirmed text on profile bubbles for Private and Tor windows as per brave/brave-core#6912 (comment).

Dark theme example Dark theme example
image image
Private window example Private window example Private window example
image image image
Tor window example Tor window example Tor window example
image image image

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/private-browsing feature/tor OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.