arlolra / ctypes-otr

js-ctypes wrapper for libotr
Mozilla Public License 2.0
29 stars 7 forks source link

Better support for convos w/ multiple sessions #45

Open arlolra opened 9 years ago

arlolra commented 9 years ago

"refreshed the private conversation" is a strange wording

cc/ @azadi @fqueze

azadi commented 9 years ago

(Another related string is the Refresh private conversation label in the padlock options.)

I see in the OTR documentation the word "refresh" is used. Can we change it to perhaps "restart"? I am not sure but "restart" seems more easy to decipher than "refresh" for some reason.

And perhaps, we can remove this string altogether:

Attempting to refresh the private conversation with

And instead just say this directly:

The OTR conversation with X has been restarted.

And if it fails, complain accordingly, but don't say "attempting" first?

arlolra commented 9 years ago

So, the point of this refreshing thing is that some transport protocols allow you to login to the same account from multiple devices simultaneously. Refreshing lets you establish OTR sessions with each device individually (a necessity since they don't share short term ratcheting keys). It then uses instance tags to indicate the intended recipient and lets the other clients drop those messages.

The docs depict a UI element we're missing; selecting to which session to send.

In that light, "restart" doesn't seem like the right term. But "refresh" is also unclear. Hmm.