<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/104667/">http://git.reviewboard.kde.org/r/104667/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 29th, 2012, 10:41 a.m., <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
</blockquote>
<p>On April 29th, 2012, noon, <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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. </pre>
</blockquote>
<p>On April 29th, 2012, 12:06 p.m., <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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!</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
<br />
<p>- Dominik</p>
<br />
<p>On April 19th, 2012, 3:02 p.m., Dominik Cermak wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Dominik Cermak.</div>
<p style="color: grey;"><i>Updated April 19, 2012, 3:02 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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?</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=294679">294679</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>contact-list-widget.h <span style="color: grey">(9688c0c5c68bd9c182cf2e978d977de09d07cfc6)</span></li>
<li>contact-list-widget.cpp <span style="color: grey">(0a1d434d9ec9bc3ea4eff4b241000e72c481b553)</span></li>
<li>main-widget.cpp <span style="color: grey">(b78cd3e433f6e81fc251e842053793db798e8011)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104667/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>