Review Request: Adds tooltips and avatar button support
Francesco Nwokeka
francesco.nwokeka at gmail.com
Fri Apr 8 00:44:53 CEST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101012/#review2488
-----------------------------------------------------------
contact-delegate.h
<http://git.reviewboard.kde.org/r/101012/#comment2157>
put a comment telling others what this does. It's name alone doesn't say that it shows the tooltip with the info
- Francesco
On April 7, 2011, 10:27 p.m., Keith Rusler wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101012/
> -----------------------------------------------------------
>
> (Updated April 7, 2011, 10:27 p.m.)
>
>
> Review request for Telepathy.
>
>
> Summary
> -------
>
> - Added support for tooltips when hovering over a contact in the list.
> - Added support for the avatar button at the top of the contact list.
> Now it will load the avatar when connecting, though I'm not sure on
> how to handle it if a protocol changed the avatar outside of telepathy.
> If the avatar is big it will resize it, so you don't have a huge tooltip
> when you hover your accounts.
>
>
> Diffs
> -----
>
> contact-delegate.h 20daa62
> contact-delegate.cpp 73f37e4
> main-widget.cpp 7bde815
>
> Diff: http://git.reviewboard.kde.org/r/101012/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Keith
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110407/a74edb23/attachment.htm
More information about the KDE-Telepathy
mailing list