> On April 30, 2014, 11:14 a.m., Martin Klapetek wrote: > > lib/chat-widget.cpp, line 796 > > <https://git.reviewboard.kde.org/r/117809/diff/2/?file=269499#file269499line796> > > > > "Please try again when the account is online." > > > > This is a bit weird wording, the account is in user's control; this > > makes it sound like "oh just wait, the account will eventually be > > online..somehow". > > > > Let's change this to "Please connect the account first and then resend". > > Thomas Pfeiffer wrote: > It depends on whether the account is offline on purpose or not. > I think we should stick to original wording when the requested state is > online (meaning that it indeed is not in the user's control), and use "Please > connect the account and then try again" ("resend" sounds like it was already > sent) when the requested state is offline (and we show the button to go > online)
Okay, maybe "Please try again when the account is connected again" to make it more clear that it's a connection problem and not some magical "sometimes it's offline, sometimes it's online" kind of thing - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117809/#review56967 ----------------------------------------------------------- On April 29, 2014, 10:58 p.m., Ahmed I. Khalil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117809/ > ----------------------------------------------------------- > > (Updated April 29, 2014, 10:58 p.m.) > > > Review request for Telepathy. > > > Bugs: 320385 > http://bugs.kde.org/show_bug.cgi?id=320385 > > > Repository: ktp-text-ui > > > Description > ------- > > Don't disable the input box when disconnected and the user sends a message a > status message is added saying "Your message was not sent because you are > offline" and the message box is not cleared. > > > Diffs > ----- > > lib/chat-widget.h 028ece4 > lib/chat-widget.cpp c4f2ea5 > > Diff: https://git.reviewboard.kde.org/r/117809/diff/ > > > Testing > ------- > > Turned the connection offline, tried to send a message, a status message is > added as expected. Then turned the status to online the message is still > there and when pressed return the message is sent normally. > > > File Attachments > ---------------- > > Message Widget with "Switch Online" button > > https://git.reviewboard.kde.org/media/uploaded/files/2014/04/29/6139c0a4-c926-4c13-98cb-017fedab3155__Disconnected_with_online_button.png > Message Widget without the "Switch Online" button > > https://git.reviewboard.kde.org/media/uploaded/files/2014/04/29/9b46448b-3337-45ff-a336-df1f3ffe125e__Disconnected_with_no_online_button.png > > > Thanks, > > Ahmed I. Khalil > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
