#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: | Sponsor9 |
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 (16)
Changed 22 months ago by
| Attachment: | 28187_screenshot.png added |
|---|
comment:1 Changed 22 months ago by
comment:2 Changed 21 months ago by
| Keywords: | TorBrowserTeam201811R added |
|---|---|
| Status: | new → needs_review |
comment:3 follow-up: 4 Changed 21 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 21 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 21 months ago by
| Attachment: | #28187.png added |
|---|
Changed 21 months ago by
| Attachment: | assets_icon.zip added |
|---|
comment:5 Changed 21 months ago by
yes, lets update it. I attached a mockup and the icon assets to replace at the hamburguer menu too. Thanks!
Changed 21 months ago by
| Attachment: | #28187-2.png added |
|---|
Changed 21 months ago by
| Attachment: | assets_icon-2.zip added |
|---|
comment:6 Changed 21 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 21 months ago by
| Attachment: | 28187_1_hamburger.png added |
|---|
comment:7 Changed 21 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 21 months ago by
| Keywords: | TorBrowserTeam201811R added; TorBrowserTeam201811 removed |
|---|---|
| Status: | needs_revision → needs_review |
comment:9 Changed 21 months ago by
| Resolution: | → fixed |
|---|---|
| Status: | needs_review → closed |
Looks good to me. Merged to master (commit 6a9a4f528d8b8254402802a7a86cd722b747a5a6).
comment:10 Changed 15 months ago by
| Sponsor: | → Sponsor9 |
|---|
Here is a patch for review: https://github.com/arthuredelstein/torbutton/commit/28187
And here is a screenshot, also for review: