> 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. > > Martin Klapetek wrote: > 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!
Until now we have a "Show/Hide Offline Users" button in the toolbar. I added submenu to the wrench-button (just to clarify). You said uniting the filtering options would be nice, so what about the following: Let the "Show/Hide Offline Users" button have more states, so there is only one button modifying the filter. * Show offline contacts * Hide offline contacts * Only show blocked contacts (or Hide unblocked contacts) (Maybe divide the last point in two, but I don't think that's a good idea: * Only show blocked contacts but hide offline ones * Only show blocked contacts, offline too) This way the only thing missing is showing blocked contacts together with the other ones. But I don't think we need this that much, because they are blocked for a reason after all. So please share your thoughts/comments. - Dominik ----------------------------------------------------------- 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
