Project

General

Profile

Feature #11210

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

Added by sajolida over 3 years ago. Updated over 3 years ago.

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

0%

Feature Branch:
feature/11210-review-user-strings
Type of work:
User interface design
Blueprint:
Starter:
Affected tool:
Onion Circuits


Related issues

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

History

#1 Updated by intrigeri over 3 years ago

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

#2 Updated by intrigeri over 3 years ago

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

#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 over 3 years ago

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

#4 Updated by intrigeri over 3 years ago

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

#5 Updated by sajolida over 3 years ago

#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 over 3 years ago

  • 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 over 3 years ago

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 over 3 years ago

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 over 3 years ago

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

#10 Updated by anonym over 3 years ago

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

#11 Updated by alant over 3 years ago

  • 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 over 3 years ago

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 over 3 years ago

#14 Updated by anonym over 3 years ago

  • Status changed from Fix committed to Resolved

Also available in: Atom PDF