Review Request 117607: New name for the ContactList plasmoid

Thomas Pfeiffer colomar at autistici.org
Thu Apr 17 08:50:10 UTC 2014


-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140417/54f4ea89/attachment-0001.html>


More information about the KDE-Telepathy mailing list