<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/102669/">http://git.reviewboard.kde.org/r/102669/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 20th, 2011, 9:46 a.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;">Pretty cool work, I'm just not sure I like the frame being turned into listview after enabling it, but then again, it's my personal feeling. Otherwise very good!</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;">Maybe it is not the best way to do it, and I've being thinking if it was better to use a normal widget for that, but in my opinion this approach has some good points in favor:
0) Saves a lot of time in coding just by adding 2 filters to the model
1) The style is consistent with the other accounts, the user will realize that it is just another account and not some kind of voodoo magic.
1b) If for some reason the style of the accounts will be changed in the future, this will be changed es well, and we won't need to change this frame to keep consistency
2) You get the same features (status, error messages, etc) handled by the delegate
3) You can disable, edit or remove the account like any other account... when you remove it it switches back to the enable frame
4) If for some weird reason you manage to get 2 local-xmpp accounts, you can handle both of them from the listview (even though the widget won't get bigger, you will get a scrollbar) therefore you will be able to fix the situation. This won't happen if you use a "static" widget.
On the other hand, maybe it is possible to change the background color from white to gray, but I'm not sure if this is a good idea...</pre>
<br />
<p>- Daniele Elmo</p>
<br />
<p>On September 19th, 2011, 11:06 p.m., Daniele Elmo Domenichelli 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 Daniele Elmo Domenichelli.</div>
<p style="color: grey;"><i>Updated Sept. 19, 2011, 11:06 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;">As discussed at Woshibon 2011!
See the screenshots...
Since the patch is quite big, you can find the git branch here:
http://quickgit.kde.org/?p=clones%2Ftelepathy-accounts-kcm%2Fddomenichelli%2Ftelepathy-accounts-kcm.git&a=shortlog&h=refs/heads/salut
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Added and removed accounts several times, uninstalled salut, edited parameters, and more</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=282198">282198</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>src/CMakeLists.txt <span style="color: grey">(c55c04622e177283007460d51593a39fccb07b55)</span></li>
<li>src/KCMTelepathyAccounts/profile-list-model.h <span style="color: grey">(63e5a16c46e6fcaa33fb4ed00d0affdb3ad8491c)</span></li>
<li>src/KCMTelepathyAccounts/profile-list-model.cpp <span style="color: grey">(c7536ab8433138273a63ddbb3fd7d37ffbd51c5f)</span></li>
<li>src/KCMTelepathyAccounts/profile-select-widget.h <span style="color: grey">(4bb92a153721b25abf44a6da52495c480fc03085)</span></li>
<li>src/KCMTelepathyAccounts/profile-select-widget.cpp <span style="color: grey">(0d5c804d25eede91b8a9ded5523e4ca9087bd8d8)</span></li>
<li>src/account-item.h <span style="color: grey">(809b93f85d111bbf13595d263faa450ff79f0073)</span></li>
<li>src/account-item.cpp <span style="color: grey">(033ac44bfed06965b8efd842d75c33fa4d8a3bb5)</span></li>
<li>src/accounts-list-model.h <span style="color: grey">(cc81c1e3e417eb8e6a35a4b8fb48205fea430452)</span></li>
<li>src/accounts-list-model.cpp <span style="color: grey">(e7bbeca9f923f21e42a06ea0a3046ed1329cc5e0)</span></li>
<li>src/kcm-telepathy-accounts.h <span style="color: grey">(6f6c3138bc39373c8b7050b2164354aabd758d07)</span></li>
<li>src/kcm-telepathy-accounts.cpp <span style="color: grey">(8d0a1cb4816055b4ddcfa8c0a377d8f1b11c0e32)</span></li>
<li>src/main-widget.ui <span style="color: grey">(1499951c28007c9166712b05df81bcb6e3ece77a)</span></li>
<li>src/salut-enable-dialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/salut-enable-dialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102669/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/102669/s/264/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/19/snapshot5_400x100.png" style="border: 1px black solid;" alt="Enable local-xmpp frame" /></a>
<a href="http://git.reviewboard.kde.org/r/102669/s/265/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/19/snapshot6_400x100.png" style="border: 1px black solid;" alt="Local-xmpp account enabled" /></a>
<a href="http://git.reviewboard.kde.org/r/102669/s/266/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/19/snapshot7_400x100.png" style="border: 1px black solid;" alt="Salut not found" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>