<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/112364/">http://git.reviewboard.kde.org/r/112364/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I would not ship this patch. I'll elaborate a bit more my comments at https://bugs.kde.org/show_bug.cgi?id=312816#c6 and I'll base my arguments on this recording I made http://kmymoney2.sourceforge.net/screencasts/112364.ogv about how this patch behaves.
1. It breaks with the current behavior of the application (auto resizing columns after data has been entered to fit the entered data)
2. It introduces bugs, I mean, setting a maxWidth of "9,999,999.00 " for the amount fields!? This is exactly what was hurting the number column and the fix should be all about removing a limitation not adding new ones</pre>
<br />
<p>- Cristian</p>
<br />
<p>On September 15th, 2013, 4:26 p.m. UTC, Allan Anderson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KMymoney.</div>
<div>By Allan Anderson.</div>
<p style="color: grey;"><i>Updated Sept. 15, 2013, 4:26 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;">If I choose to use a complex system for check numbers, such that the whole number is not visible,
the only way I have available is to stretch the whole window. However, even that doesn't help,
as the whole of the increase is grabbed by the Details column. I accept that it is likely that that
column is going to need to be the widest. Then, why are the Payment and Deposit columns twice the
width of the Balance column, when that column may be likely to have the greatest value? Ditto for
the Date.
This fix allows modification of column widths, but also resizes the individual columns to more suitable widths.
I found that Thomas had started to implement something similar some while ago, so I have built upon and expanded that.
I found that the edit widgets were particularly troublesome, in failing to appear/disappear with the show() and hide()
methods, which I'd previously found when last in this area. Then, when the screen was being resized, they flickered
more than acceptable. Eventually, where necessary, I resorted to zeroing/resetting the height instead, which resolved
the issue, although with some complication.</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;">Extensive editing of sample files, and changing back and forth between different activity types, which tended to show
problem areas. atype run.
</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=312816">312816</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=322768">322768</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>kmymoney/dialogs/transactioneditor.h <span style="color: grey">(f07dafb)</span></li>
<li>kmymoney/dialogs/transactioneditor.cpp <span style="color: grey">(39049cf)</span></li>
<li>kmymoney/views/kgloballedgerview.cpp <span style="color: grey">(2057b47)</span></li>
<li>kmymoney/widgets/register.h <span style="color: grey">(eebe78d)</span></li>
<li>kmymoney/widgets/register.cpp <span style="color: grey">(1bdf5bd)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/112364/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>