----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112043/#review37629 -----------------------------------------------------------
app/chat-window.cpp <http://git.reviewboard.kde.org/r/112043/#comment27801> I'm not sure you want to remove this line. (the first of these 2) This will stop it updating the window icon I think. - David Edmundson On Aug. 12, 2013, 11:56 p.m., Daniel Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112043/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2013, 11:56 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > When a user is talking to multiple people, it is confusing to have 'Typing... > Bob', when it's actually Fred in another tab who is typing. > > The presence of the both the chat tab status-icon change combined with the > tab name text colour change is sufficient to denote activity and removes this > confusion which is exacerbated with the more chat tabs that you have open. > > > Diffs > ----- > > app/chat-window.cpp d829125 > app/chat-window.h 3a5b6ba > > Diff: http://git.reviewboard.kde.org/r/112043/diff/ > > > Testing > ------- > > Tabs still change icon and colour upon user typing/message received. > > > Thanks, > > Daniel Cohen > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
