<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 />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On November 24th, 2012, 11:51 a.m., <b>Alvaro Soliverez</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;">Other than that, it's ok for pushing.
Do you have push access? If you don't, it's time to ask for your account.
Fill me in as your supporter.</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;">No, I do not have push access.

Where would I apply for it?
(Having asked this I am aware that my coding-capabilities for KDE are rather limited, not to talk about my seldom use of git.)</pre>
<br />








<p>- Marko</p>


<br />
<p>On November 24th, 2012, 11:42 a.m., Marko Käning 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 KMymoney.</div>
<div>By Marko Käning.</div>


<p style="color: grey;"><i>Updated Nov. 24, 2012, 11:42 a.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;">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> </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/851/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/11/24/KMM-homepage-online-status_400x100.png" style="border: 1px black solid;" alt="online-status columns in "Assets and Liabilities Summary", "Preferred Accounts" and "Payment Accounts" tables" /></a>

</div>


  </td>
 </tr>
</table>








  </div>
 </body>
</html>