> On April 12, 2011, 2:52 p.m., Dario Freddi wrote: > > lib/chat-widget.cpp, line 682 > > <http://git.reviewboard.kde.org/r/101105/diff/1/?file=14226#file14226line682> > > > > This message does not look really explainatory to me. Maybe David can > > suggest something better?
Yeah, I know it's no good. - Lasse ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101105/#review2591 ----------------------------------------------------------- On April 12, 2011, 3:10 p.m., Lasse Liehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101105/ > ----------------------------------------------------------- > > (Updated April 12, 2011, 3:10 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > I'm not sure about the message, but there should be some. Ideas about a > better message are really appreciated. I'm also not sure if the line: > statusMessage.setService(d->channel->connection()->protocolName()); > is good or not. I don't know what "service" is used for. > > > This addresses bug 270725. > http://bugs.kde.org/show_bug.cgi?id=270725 > > > Diffs > ----- > > lib/chat-widget.h fdf5ccd > lib/chat-widget.cpp af7edff > > Diff: http://git.reviewboard.kde.org/r/101105/diff > > > Testing > ------- > > 1. Open a text chat with someone > 2. Chat > 3. Go offline (don't close the chat window/tab) > 4. Send message line edit and button are disabled, informational message is > shown > > > Thanks, > > Lasse > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
