----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100940/#review2151 -----------------------------------------------------------
Ship it! Some small code polish suggests, otherwise looks good. lib/channel-contact-model.cpp <http://git.reviewboard.kde.org/r/100940/#comment1777> These could use some code styling (align the '&') lib/chat-widget.ui <http://git.reviewboard.kde.org/r/100940/#comment1776> Shouldn't we make these numbers look..uhm..pretty? :) (kinda irrelevant really...just a thought) - Martin On March 24, 2011, 7:52 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100940/ > ----------------------------------------------------------- > > (Updated March 24, 2011, 7:52 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > If there are more than 1 other contact in a group chat show a list of these > users in a sidebar. > > > Diffs > ----- > > lib/CMakeLists.txt 5dc3f74aede922597ced7a7a2229fa141b3372ea > lib/channel-contact-list.h 37f29e6305c96a1aa9f3a30a2eabb141139f33f5 > lib/channel-contact-list.cpp 2eccee0a852e47d90a74a518adcc9260e1c8cef7 > lib/channel-contact-model.h PRE-CREATION > lib/channel-contact-model.cpp PRE-CREATION > lib/chat-widget.cpp 6ea639b0b3696030bd76bba219522abf70aef400 > lib/chat-widget.ui 173e1151bf04c8573525ff193adffb7129df808a > > Diff: http://git.reviewboard.kde.org/r/100940/diff > > > Testing > ------- > > Opened a chat, saw that some people were there. > > > Thanks, > > David > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
