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

redo Tor newtab UI to remove Tor logo #14121

Closed
diracdeltas opened this issue May 14, 2018 · 4 comments
Closed

redo Tor newtab UI to remove Tor logo #14121

diracdeltas opened this issue May 14, 2018 · 4 comments

Comments

@diracdeltas
Copy link
Member

diracdeltas commented May 14, 2018

also need to add '/' at the end of the url: https://torproject.org/

test plan:

  1. open new tor tab
  2. it shouldn't show the Tor onion logo, instead showing an eye logo
@diracdeltas diracdeltas added the needs-mockup A feature which needs design mockup to be implemented. label May 14, 2018
@diracdeltas diracdeltas added this to the 0.23.x (Developer Channel) milestone May 14, 2018
diracdeltas added a commit that referenced this issue May 17, 2018
diracdeltas added a commit that referenced this issue May 17, 2018
@diracdeltas
Copy link
Member Author

fixed by a93cfd4 and 5a73788

diracdeltas added a commit that referenced this issue May 22, 2018
and add slash to end of torproject URL

fix #14121
diracdeltas added a commit that referenced this issue May 22, 2018
diracdeltas added a commit that referenced this issue Jun 8, 2018
and add slash to end of torproject URL

fix #14121
diracdeltas added a commit that referenced this issue Jun 8, 2018
diracdeltas added a commit that referenced this issue Jun 12, 2018
and add slash to end of torproject URL

fix #14121
diracdeltas added a commit that referenced this issue Jun 12, 2018
petemill pushed a commit that referenced this issue Jun 15, 2018
and add slash to end of torproject URL

fix #14121
petemill pushed a commit that referenced this issue Jun 15, 2018
@kjozwiak
Copy link
Member

@diracdeltas @bradleyrichter there isn't much information here for QA. I'm assuming the old design included the Tor logo under about:newtab for PT's which has been removed. Can you please confirm that this is the current design:

screen shot 2018-06-20 at 12 00 48 pm

Once confirmed, I'll quickly create a test plan in the original issue and tag this issue as QA/test-plan-specified.

bsclifton pushed a commit that referenced this issue Jun 21, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 21, 2018
bsclifton pushed a commit that referenced this issue Jun 21, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 21, 2018
@diracdeltas diracdeltas removed QA/test-plan-required needs-mockup A feature which needs design mockup to be implemented. labels Jun 21, 2018
@rossmoody
Copy link
Contributor

@kjozwiak I've been working on this screen quite a bit and can confirm there is no Tor branding elements anywhere except for the fav.ico in regular type reading "Tor" in the tab when it's enabled

bsclifton pushed a commit that referenced this issue Jun 22, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 22, 2018
@kjozwiak
Copy link
Member

kjozwiak commented Jun 22, 2018

Verified on macOS 10.13.4 x64 using the following build:

  • 0.23.12 88f6f07
  • muon: 7.1.1
  • libchromiumcontent: 67.0.3396.87

Verified on Ubuntu 17.10 x64

  • 0.23.14 - f4da855
  • Muon - 7.1.1
  • libchromiumcontent - 67.0.3396.87

diracdeltas added a commit that referenced this issue Jun 22, 2018
and add slash to end of torproject URL

fix #14121
diracdeltas added a commit that referenced this issue Jun 22, 2018
bsclifton pushed a commit that referenced this issue Jun 23, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 23, 2018
bsclifton pushed a commit that referenced this issue Jun 25, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 25, 2018
bsclifton pushed a commit that referenced this issue Jun 26, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 26, 2018
bsclifton pushed a commit that referenced this issue Jun 27, 2018
and add slash to end of torproject URL

fix #14121
bsclifton pushed a commit that referenced this issue Jun 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.