Feature #11210

Review user-visible strings of Onion Circuits and Tor Status extension

Added by sajolida 2016-03-09 12:57:12 . Updated 2016-06-08 01:31:02 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2016-03-09
Due date:
% Done:

0%

Feature Branch:
feature/11210-review-user-strings
Type of work:
User interface design
Blueprint:

Starter:
Affected tool:
Onion Circuits
Deliverable for:


Subtasks


Related issues

Has duplicate Tails - Feature #11161: Have UX people review Onion Circuits and Tor Status extension strings Resolved 2016-02-23
Blocked by Tails - Feature #11443: Upgrade Onion Circuits to 0.4 Resolved 2016-05-18

History

#1 Updated by intrigeri 2016-03-09 13:47:32

  • blocks Feature #11161: Have UX people review Onion Circuits and Tor Status extension strings added

#2 Updated by intrigeri 2016-03-25 23:26:02

  • Subject changed from Review user-visible strings of Onion Circuits to Review user-visible strings of Onion Circuits and Tor Status extension

Feature #11160 “Have Onion Circuits translatable in Transifex” is currently (transitively) blocked by this ticket. On March 9 you said there was no ETA for this task. So I’m tempted to stop blocking on this, and push these new (currently non-translatable) strings to Transifex, because 1. Onion Circuits & Tor Status have very few strings, so even if we waste some translators’ time it’s no huge deal (iterate!); 2. localized suboptimal strings would be better than suboptimal strings in English, for most users; 3. waiting a month for a string review was enough, before I go ahead and have the strings translated; 4. I want to allow you to take it easy. Do you agree with this proposed course of action?

#3 Updated by intrigeri 2016-03-25 23:27:41

  • blocked by deleted (Feature #11161: Have UX people review Onion Circuits and Tor Status extension strings)

#4 Updated by intrigeri 2016-03-25 23:27:51

  • has duplicate Feature #11161: Have UX people review Onion Circuits and Tor Status extension strings added

#5 Updated by sajolida 2016-03-29 17:12:00

> Feature #11160 “Have Onion Circuits translatable in Transifex” is currently
> (transitively) blocked by this ticket.

I didn’t realize this until now, sorry! I worked on this today and will
show you stuff soon. I got tempted to modify the code a tiny little bit
so I’ll need a review from Alan as well.

#6 Updated by sajolida 2016-03-30 08:15:46

  • Assignee changed from sajolida to alant
  • Target version set to Tails_2.3
  • QA Check set to Ready for QA
  • Feature Branch set to feature/11210-review-user-strings

Checked with Alan, and this should be feasible for 2.3 — considering them as UX bugs :)

#7 Updated by sajolida 2016-04-14 09:58:41

All this was reviewed with Alan already and I fixed the infobar issue with d1a7ab6. I think this branch is ready for merge now.

#8 Updated by intrigeri 2016-04-14 18:00:09

> All this was reviewed with Alan already and I fixed the infobar issue with d1a7ab6. I think this branch is ready for merge now.

Cool! You left Alan as assignee, does that mean we should wait for him to review this again, or?

#9 Updated by sajolida 2016-04-22 05:52:16

Yes, for the change in the Infobar code in d1a7ab6.

#10 Updated by anonym 2016-05-08 05:10:31

  • Target version changed from Tails_2.3 to Tails_2.4

#11 Updated by alant 2016-05-15 07:15:29

  • Status changed from Confirmed to Fix committed
  • Assignee deleted (alant)
  • QA Check changed from Ready for QA to Pass

I merged the branch in master (which included fixing some conflicts) and fixed an issue it introduced.

#12 Updated by intrigeri 2016-05-18 08:47:09

So, that’s part of the 0.4 upstream release, yeah :) Let’s try to get it into Tails 2.4 so that this ticket’s metadata becomes correct.

#13 Updated by intrigeri 2016-05-18 09:30:50

#14 Updated by anonym 2016-06-08 01:31:02

  • Status changed from Fix committed to Resolved