----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109033/#review27695 -----------------------------------------------------------
Ship it! KTp/Declarative/contact-list.cpp <http://git.reviewboard.kde.org/r/109033/#comment20751> We shouldn't change a declarative plugin to match what one of the user's of the plugin wants. This changes the behaviour of ktp-active and we should be thinking about making things general. However, I think that won't ever need to delegate, so I'll allow this patch. Also note that I want to get rid of this class (see my RB proposals online already) however I'll make sure my version also has this parameter changed if/when it gets merged. - David Edmundson On Feb. 19, 2013, 1:28 a.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109033/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2013, 1:28 a.m.) > > > Review request for Telepathy and David Edmundson. > > > Description > ------- > > This patch makes it possible to start conversations from the contact list > plasmoid. I think it's a good start into making out of the ktp-chat a useful > tool. > It's not cool to ask people to ping you if you want to talk with them on the > ktp-chat for the first time... ;) > > > Diffs > ----- > > KTp/Declarative/contact-list.cpp 58da5ab > > Diff: http://git.reviewboard.kde.org/r/109033/diff/ > > > Testing > ------- > > It works, although I'm unsure if it does what I think it does > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
