Opened 4 months ago

Closed 3 months 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 4 months ago.
#28187.png​ (227.9 KB) - added by antonela 3 months ago.
assets_icon.zip​ (7.3 KB) - added by antonela 3 months ago.
#28187-2.png​ (220.6 KB) - added by antonela 3 months ago.
assets_icon-2.zip​ (8.2 KB) - added by antonela 3 months ago.
28187_1_hamburger.png​ (30.2 KB) - added by arthuredelstein 3 months ago.

Download all attachments as: .zip

Change History (15)

Changed 4 months ago by arthuredelstein

Attachment: 28187_screenshot.png​ added

comment:1 Changed 4 months 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 3 months ago by gk

Keywords: TorBrowserTeam201811R added
Status: new → needs_review

comment:3 Changed 3 months 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 3 months ago by gk

Keywords: TorBrowserTeam201811 added; TorBrowserTeam201811R removed
Status: needs_review → needs_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 3 months ago by antonela

Attachment: #28187.png​ added

Changed 3 months ago by antonela

Attachment: assets_icon.zip​ added

comment:5 Changed 3 months ago by antonela

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

Changed 3 months ago by antonela

Attachment: #28187-2.png​ added

Changed 3 months ago by antonela

Attachment: assets_icon-2.zip​ added

comment:6 Changed 3 months 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 3 months ago by arthuredelstein

Attachment: 28187_1_hamburger.png​ added

comment:7 Changed 3 months 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 3 months ago by arthuredelstein (previous) (diff)

comment:8 Changed 3 months ago by arthuredelstein

Keywords: TorBrowserTeam201811R added; TorBrowserTeam201811 removed
Status: needs_revision → needs_review

comment:9 Changed 3 months ago by gk

Resolution: → fixed
Status: needs_review → closed

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

Note: See TracTickets for help on using tickets.