> On Sept. 6, 2013, 12:35 a.m., Aleix Pol Gonzalez wrote: > > I think it could be useful... but what's the use-case? > > > > Maybe we should have one more generic that displays the standard > > information about the subcontacts instead of just the IM? Like e-mail or > > phone... > > > > It doesn't seem like you require ktp anyway for what you're showing, right? > > David Edmundson wrote: > >what's the use-case? > > To see IM contacts from inside person viewer, to copy + paste addresses, > etc. > > >Maybe we should have one more generic that displays the standard > information about the subcontacts instead of just the IM? Like e-mail or > phone... > > It does! > The entire right hand side of person-details-view is built on a plugin > system. It's the same widget being shown here. > > This is one of the plugins it requires KTp because it shows presence. In > addition at some point I want it to show blocked-state, subscription state > the account and have all the actions attached. >
Ok, so if we need to have a separate plugin for KTp then it's ok. I'll give you a "ship it" once the issues are solved. (mainly the 16 thing XD) - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112549/#review39445 ----------------------------------------------------------- On Sept. 6, 2013, 12:05 a.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112549/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2013, 12:05 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > Add kpeople plugin to show IM contacts in show details view > > > Diffs > ----- > > kpeople/CMakeLists.txt 10c3758aa0fd54177595a69afd4eb899664b4193 > kpeople/uiplugins/CMakeLists.txt PRE-CREATION > kpeople/uiplugins/imdetailswidget.h PRE-CREATION > kpeople/uiplugins/imdetailswidget.cpp PRE-CREATION > kpeople/uiplugins/imdetailswidgetplugin.desktop PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/112549/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > > > http://git.reviewboard.kde.org/media/uploaded/files/2013/09/06/snapshot2_1.png > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
