<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/107364/">http://git.reviewboard.kde.org/r/107364/</a>
</td>
</tr>
</table>
<br />
<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 KMymoney.</div>
<div>By Marko Käning.</div>
<p style="color: grey;"><i>Updated Nov. 23, 2012, 11:30 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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 now also fixes the display problems of the preferred and payment accounts tables.</pre>
</td>
</tr>
</table>
<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;">Homepage: extend accounts tables with additional columns to reflect online-status
* green check-mark -> for accounts with matching file and online balance
* red warning -> for accounts with unequal file and online balance
* a minus -> for accounts without a valid online balance
The status column - being right now in between account name and balance - could also be moved in front of the account name or after the balance, depending on what usability dictates...
This feature is opt-in configurable from application preferences for users who have no need for this. (Thanks to David Houlden for his feedback!)
I've posted this to ReviewBoard to get feedback from users but also from core-devs whether the implementation of such a feature would make sense.</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;">built and ran w/o problems</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kmymoney/dialogs/settings/ksettingshomedecl.ui <span style="color: grey">(df9b95c80bea637bfe8f644c1797e367cdfbba03)</span></li>
<li>kmymoney/kmymoney.kcfg <span style="color: grey">(5b6658615c7d7e74b71aff4d4a2bf7cc4a322823)</span></li>
<li>kmymoney/views/khomeview.cpp <span style="color: grey">(498e4a5d568500ab145f11e654f2e482dba652b4)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107364/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/107364/s/836/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/11/18/KMM-homepage-online-status_400x100.png" style="border: 1px black solid;" alt="Accounts view with new "status" columns" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>