----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108927/#review27370 -----------------------------------------------------------
config/appearance-config.cpp <http://git.reviewboard.kde.org/r/108927/#comment20592> You shouldn't hide the /me messages, therefore also the demo chat should reflect this ("Ted Example waves." is a /me message. Anyway is the "if" necessary? Is it possible just to let the chat widget handle that? I'd like to see the demo chat just like a sequence of events and see exactly what I would see in the chat ui (but perhaps it's not possible) lib/adium-theme-view.h <http://git.reviewboard.kde.org/r/108927/#comment20593> The config for group chats is being split into a separate tab (see http://git.reviewboard.kde.org/r/108571/ ). Therefore I'd suggest to have just one bool for now, instead of an enum. Later we will have 2 different bools in single/group chats. Otherwise we will have to handle the migration of the setting. Anyway I would like to hear David's opinion - Daniele E. Domenichelli On Feb. 13, 2013, 2:29 p.m., Andrea Scarpino wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108927/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2013, 2:29 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Implement a way to turn the presence-changes message off based on last David > comment in the bug report. > So, this adds a combobox in the Chat Window Appareance tab. > > > This addresses bug 287399. > http://bugs.kde.org/show_bug.cgi?id=287399 > > > Diffs > ----- > > config/appearance-config.h 5b6c83d > config/appearance-config.cpp a88dd02 > config/appearance-config.ui 6a040db > lib/adium-theme-view.h 732cf71 > lib/adium-theme-view.cpp 12e0b02 > lib/chat-widget.cpp 60efede > > Diff: http://git.reviewboard.kde.org/r/108927/diff/ > > > Testing > ------- > > No more presence-changes messages :) > > > Thanks, > > Andrea Scarpino > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
