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)
Change History (15)
Changed 6 weeks ago by
| Attachment: | 28187_screenshot.png added |
|---|
comment:1 Changed 6 weeks ago by
comment:2 Changed 5 weeks ago by
| Keywords: | TorBrowserTeam201811R added |
|---|---|
| Status: | new → needs_review |
comment:3 follow-up: 4 Changed 5 weeks 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 5 weeks 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 5 weeks ago by
| Attachment: | #28187.png added |
|---|
Changed 5 weeks ago by
| Attachment: | assets_icon.zip added |
|---|
comment:5 Changed 5 weeks ago by
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
| Attachment: | #28187-2.png added |
|---|
Changed 5 weeks ago by
| Attachment: | assets_icon-2.zip added |
|---|
comment:6 Changed 5 weeks 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 5 weeks ago by
| Attachment: | 28187_1_hamburger.png added |
|---|
comment:7 Changed 5 weeks 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 5 weeks ago by
| Keywords: | TorBrowserTeam201811R added; TorBrowserTeam201811 removed |
|---|---|
| Status: | needs_revision → needs_review |
comment:9 Changed 5 weeks 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: