Review Request 111097: Use KTp::ContactGridDialog for Contact applet configuration

[David Edmundson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111097/#review34587
-----------------------------------------------------------

Ship it!


There are comments I want fixing.


contact/CMakeLists.txt
<http://git.reviewboard.kde.org/r/111097/#comment25388>

    ...and this.



contact/CMakeLists.txt
<http://git.reviewboard.kde.org/r/111097/#comment25387>

    This can be deleted too now, right?



contact/src/telepathy-contact.cpp
<http://git.reviewboard.kde.org/r/111097/#comment25386>

    Is this needed?
    
    If so that means the filter is broken.


- David Edmundson


On June 18, 2013, 2:34 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111097/
> -----------------------------------------------------------
> 
> (Updated June 18, 2013, 2:34 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Use Ktp::ContactGridDialog instead of own UI, since the grid is known to work and is sort of standard UI for contact selection in KTp.
> 
> The main advantage of of ContactGridDialog is that it has a search field, which is I think more useful than 'show online/offline' toggle, since you usually know who you want to add to the applet when you are configuring it, so you will more probably just want to type contact's name and click, instead of scrolling through a list and strcmp()'ing contact names in head :-)
> 
> 
> Diffs
> -----
> 
>   contact/CMakeLists.txt a94a864 
>   contact/src/applet_config.h 772b7cf 
>   contact/src/applet_config.cpp 52fff50 
>   contact/src/applet_config.ui 1b51c46 
>   contact/src/telepathy-contact.h 6d4505f 
>   contact/src/telepathy-contact.cpp 9db8217 
> 
> Diff: http://git.reviewboard.kde.org/r/111097/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130618/78d7d53a/attachment-0001.html>