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

Private Tabs with Tor feature needs to be labeled as beta #14541

Closed
davidtemkin opened this issue Jun 25, 2018 · 15 comments
Closed

Private Tabs with Tor feature needs to be labeled as beta #14541

davidtemkin opened this issue Jun 25, 2018 · 15 comments

Comments

@davidtemkin
Copy link

davidtemkin commented Jun 25, 2018

Test plan

New private tab page should match this spec:
image

Original issue details

On the new private tab screen, the "Make this tab much more private with Tor" feature needs to be visually labeled as "beta" prior to the release of 0.23.x. A possible model for this is how Sync preferences are now labeled.

@rossmoody
Copy link
Contributor

image

@bradleyrichter @davidtemkin How do we feel about this solve? If it's a go, I can very quickly hop in there and replace this svg.

@davidtemkin
Copy link
Author

I like the treatment. It may be somewhat easy to miss because of color (e.g. I had to look for it for a second); what do you think? We don't need to scream "DON'T USE THIS" but it should be visible enough.

@rossmoody
Copy link
Contributor

I agree. Black? Still sits in the illustration but more prominent.

image

@davidtemkin
Copy link
Author

Works for me. @bradleyrichter ?

@bradleyrichter
Copy link
Contributor

It works but adding it to the title string would add to it's recognition. If we don't need that much, then let's call it done.

@rossmoody
Copy link
Contributor

Like this?
image

@bradleyrichter
Copy link
Contributor

@rossmoody Probably on the right end instead?

@rossmoody
Copy link
Contributor

image

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Jun 25, 2018 via email

@petemill
Copy link
Member

ok I'll get it in

@NumDeP
Copy link

NumDeP commented Jun 25, 2018

Are these images just a simple mock-up like paint and not Brave's actual UI model? I only ask because it will bug me a lot not seeing that toggle aligned with DDG's toggle :)

@bradleyrichter
Copy link
Contributor

@petemill will make sure they are in alignment . : )

@rossmoody
Copy link
Contributor

rossmoody commented Jun 25, 2018

Sharp eye, these were quick mockups in sketch from a dated file. The toggles will align

@bsclifton
Copy link
Member

Fixed with #14555

@btlechowski
Copy link
Contributor

btlechowski commented Jun 26, 2018

Verified on Ubuntu 17.10 x64

  • 0.23.18 - c101002
  • Muon - 7.1.2
  • libchromiumcontent - 67.0.3396.87

Verified on Windows 10 x64
• 0.23.18 c101002
• Muon 7.1.2
• libchromiumcontent: 67.0.3396.87

Verified with macOS 10.12.6 using

  • 0.23.18 c101002
  • Muon 7.1.2
  • libchromiumcontent 67.0.3396.87

screen shot 2018-06-26 at 8 59 26 am

screen shot 2018-06-26 at 8 59 44 am

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