[tor-bugs] #24309 [Applications/Tor Browser]: Activity 4.1: Improve how circuits are displayed to the user

[Tor Bug Tracker & Wiki]

#24309: Activity 4.1: Improve how circuits are displayed to the user
-------------------------------------------+------------------------------
 Reporter:  isabela                        |          Owner:  tbb-team
     Type:  defect                         |         Status:  needs_review
 Priority:  High                           |      Milestone:
Component:  Applications/Tor Browser       |        Version:
 Severity:  Normal                         |     Resolution:
 Keywords:  ux-team, TorBrowserTeam201805  |  Actual Points:
Parent ID:                                 |         Points:
 Reviewer:  antonela                       |        Sponsor:
-------------------------------------------+------------------------------

Comment (by arthuredelstein):

 Replying to [comment:62 gk]:
 > Okay, I cherry-picked ef3476c5068d8597084c781a25792ce9ccef378c and fixed
 two typos in a follow-up commit
 (a3447ddb13c279e7e4c384610b98a045e7f932bb). Thanks, great work!
 >
 > What's the idea behind the "const-ification" in the other commit? FWIW:
 I am not sure whether this is actually bound to this bug (given you used
 the same bug number).

 Something igt0 suggested when he reviewed the patch. Technically I should
 be using `const` often where I am using `let`. Maybe it's better to ignore
 this patch for now and we can decide whether to constify the whole
 codebase at some point.

 > As said in my previous comment another thing open for this ticket is
 adapting the changes to ESR 60.
 This is happening in #26100. I with my patch in the previous comment plus
 the patches we will be proposing for #26100, I expect the circuit display
 will work.

 > Additionally, we might want to open a new ticket for my point 1 in
 comment:57.

 I opened #26174.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/24309#comment:64>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online