Review Request: Make it possible to filter for blocked/unblocked contacts

[Martin Klapetek]

> On April 29, 2012, 10:41 a.m., David Edmundson wrote:
> > Ship It!
> 
> Martin Klapetek wrote:
>     Wait, let's think about it little bit. The code is fine, however we have controls of the list in the toolbar (or the settings menu). Now we add another set of (filter) controls into context menu. I'm not convinced this is a good idea. I think we could have an option to select what you want to display - "Show me all my offline friends and blocked too" || "Don't show offline contacts but still show blocked". Unite the filtering options and not put them all over the place.

Speaking with my release manager hat on now - forget this for now, there's no time for it to be done in 0.4 and it's good enough. We can rethink this for 0.5 (and I have a plan to review some stuff).

So ship it!


- Martin


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


On April 19, 2012, 3:02 p.m., Dominik Cermak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104667/
> -----------------------------------------------------------
> 
> (Updated April 19, 2012, 3:02 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> This adds a menuentry to contact-list where you can set which contacts should be shown:
>  * Show all contacts [that's how it is until now]
>  * Show unblocked contacts [default]
>  * Show blocked contacts
> 
> One annoying thing is that e.g. groups which have no contacts because of this filter still show up, together with the online contacts/total contacts count. This is true for all filters so maybe it's not such a big issue. Comments are welcome.
> 
> PS: I implemented it in the way the rest of contact-list is implemented, but should I use the same approach as in my last review (enum conversation) instead or let it this way?
> 
> 
> This addresses bug 294679.
>     http://bugs.kde.org/show_bug.cgi?id=294679
> 
> 
> Diffs
> -----
> 
>   contact-list-widget.h 9688c0c5c68bd9c182cf2e978d977de09d07cfc6 
>   contact-list-widget.cpp 0a1d434d9ec9bc3ea4eff4b241000e72c481b553 
>   main-widget.cpp b78cd3e433f6e81fc251e842053793db798e8011 
> 
> Diff: http://git.reviewboard.kde.org/r/104667/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dominik Cermak
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120429/c7ae8c6f/attachment.html>