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)
Change History (15)
Changed 4 months ago by
Attachment: | 28187_screenshot.png added |
---|
comment:1 Changed 4 months ago by
comment:2 Changed 3 months ago by
Keywords: | TorBrowserTeam201811R added |
---|---|
Status: | new → needs_review |
comment:3 follow-up: 4 Changed 3 months ago by
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 Changed 3 months ago by
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
Attachment: | #28187.png added |
---|
Changed 3 months ago by
Attachment: | assets_icon.zip added |
---|
comment:5 Changed 3 months ago by
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
Attachment: | #28187-2.png added |
---|
Changed 3 months ago by
Attachment: | assets_icon-2.zip added |
---|
comment:6 Changed 3 months ago by
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
Attachment: | 28187_1_hamburger.png added |
---|
comment:7 Changed 3 months ago by
Here's a revised patch including the new "new circuit" icon:
https://github.com/arthuredelstein/torbutton/commit/28187+1
Screenshot:
comment:8 Changed 3 months ago by
Keywords: | TorBrowserTeam201811R added; TorBrowserTeam201811 removed |
---|---|
Status: | needs_revision → needs_review |
comment:9 Changed 3 months ago by
Resolution: | → fixed |
---|---|
Status: | needs_review → closed |
Looks good to me. Merged to master
(commit 6a9a4f528d8b8254402802a7a86cd722b747a5a6).
Here is a patch for review: https://github.com/arthuredelstein/torbutton/commit/28187
And here is a screenshot, also for review: