----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117607/#review55943 -----------------------------------------------------------
Even though it's redundant to the name, I'd still add "Instant messaging" to the description: "Displays your instant messaging presence and contacts" We'll probably have to change both name and description again if we want to integrate all contacts (including email) in the Plasmoids as well at some point, but I think that's okay. - Thomas Pfeiffer On April 17, 2014, 12:55 a.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117607/ > ----------------------------------------------------------- > > (Updated April 17, 2014, 12:55 a.m.) > > > Review request for Telepathy. > > > Repository: ktp-desktop-applets > > > Description > ------- > > During the sprint, we decided that the ContactList plasmoid would unify > presence in it, and now it's working like this. > > This patch suggests a rename of the plasmoid and comment, describing the new > responsabilities of the component. > > > Diffs > ----- > > contactlist/org.kde.ktp-contactlist/metadata.desktop.cmake 502812e > > Diff: https://git.reviewboard.kde.org/r/117607/diff/ > > > Testing > ------- > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
