Opened 7 weeks ago

Closed 5 weeks ago

#28187 closed defect (fixed)

Change Tor Circuit display icon to an onion

Reported by: arthuredelstein Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-usability, ux-team, TorBrowserTeam201811R
Cc: antonela Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Right now the Tor Circuit display shows a "relay" icon, but we'd like to change it to an onion to make it more clearly a "Tor" feature. The onion icon is the same one we use in torbutton.

Child Tickets

Attachments (6)

28187_screenshot.png (41.3 KB) - added by arthuredelstein 6 weeks ago.
#28187.png (227.9 KB) - added by antonela 5 weeks ago.
assets_icon.zip (7.3 KB) - added by antonela 5 weeks ago.
#28187-2.png (220.6 KB) - added by antonela 5 weeks ago.
assets_icon-2.zip (8.2 KB) - added by antonela 5 weeks ago.
28187_1_hamburger.png (30.2 KB) - added by arthuredelstein 5 weeks ago.

Download all attachments as: .zip

Change History (15)

Changed 6 weeks ago by arthuredelstein

Attachment: 28187_screenshot.png added

comment:1 Changed 6 weeks ago by arthuredelstein

Here is a patch for review: https://github.com/arthuredelstein/torbutton/commit/28187

And here is a screenshot, also for review:


comment:2 Changed 5 weeks ago by gk

Keywords: TorBrowserTeam201811R added
Status: newneeds_review

comment:3 Changed 5 weeks ago by antonela

Arthur! looks awesome!

The main aim with this iteration is to make clear that the circuit is made inside the Tor network. For non-tech-savvy users, it will help on recognize the relationship with the network.

Should we also update the icon at the hamburger menu?

comment:4 in reply to:  3 Changed 5 weeks ago by gk

Keywords: TorBrowserTeam201811 added; TorBrowserTeam201811R removed
Status: needs_reviewneeds_revision

Replying to antonela:

Arthur! looks awesome!

The main aim with this iteration is to make clear that the circuit is made inside the Tor network. For non-tech-savvy users, it will help on recognize the relationship with the network.

Should we also update the icon at the hamburger menu?

Yes. If we update the icon in one place, then we should update it at the other one as well.

Changed 5 weeks ago by antonela

Attachment: #28187.png added

Changed 5 weeks ago by antonela

Attachment: assets_icon.zip added

comment:5 Changed 5 weeks ago by antonela

yes, lets update it. I attached a mockup and the icon assets to replace at the hamburguer menu too. Thanks!

Changed 5 weeks ago by antonela

Attachment: #28187-2.png added

Changed 5 weeks ago by antonela

Attachment: assets_icon-2.zip added

comment:6 Changed 5 weeks ago by antonela

Arthur highlighted to me that the tab behavior when users click for a [New Circuit] is different than asking for a new tab. True. The real behavior is that the circuit and the page, reload. Maybe the v2 works better :)
Mockup and assets attached.

Changed 5 weeks ago by arthuredelstein

Attachment: 28187_1_hamburger.png added

comment:7 Changed 5 weeks ago by arthuredelstein

Here's a revised patch including the new "new circuit" icon:

https://github.com/arthuredelstein/torbutton/commit/28187+1

Screenshot:


Last edited 5 weeks ago by arthuredelstein (previous) (diff)

comment:8 Changed 5 weeks ago by arthuredelstein

Keywords: TorBrowserTeam201811R added; TorBrowserTeam201811 removed
Status: needs_revisionneeds_review

comment:9 Changed 5 weeks ago by gk

Resolution: fixed
Status: needs_reviewclosed

Looks good to me. Merged to master (commit 6a9a4f528d8b8254402802a7a86cd722b747a5a6).

Note: See TracTickets for help on using tickets.