[Kmymoney-devel] Review Request 112364: BUG:312816 - Implement resizing of ledger Number column (and others)., and Interest category and amount disappear when new fee entered in Dividend.

[Cristian Oneț]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112364/#review39691
-----------------------------------------------------------



kmymoney/dialogs/investactivities.cpp
<http://git.reviewboard.kde.org/r/112364/#comment29234>

    Could you add a little comment on why do you need to set the rowHeight later to this value?



kmymoney/dialogs/investactivities.cpp
<http://git.reviewboard.kde.org/r/112364/#comment29235>

    Also I don't see a reason why the timeouts passed to QTimer::signleShot vary that much, 5, 100, 150, is there a reason for it?



kmymoney/dialogs/investactivities.cpp
<http://git.reviewboard.kde.org/r/112364/#comment29236>

    Is there a reason why the show of "shares" and "price" was move out from the for loop?



kmymoney/dialogs/investactivities.cpp
<http://git.reviewboard.kde.org/r/112364/#comment29237>

    Same questions as Buy::showWidgets()



kmymoney/dialogs/investtransactioneditor.h
<http://git.reviewboard.kde.org/r/112364/#comment29232>

    Is register really an 'int'?



kmymoney/dialogs/transactioneditor.h
<http://git.reviewboard.kde.org/r/112364/#comment29238>

    Is making this public a good idea?


Another nice thing would be to have a separate patch for each individual functionality. Right now we have two things here 1 - ledger resizing and 2 - investment editor fix for widgets that are visible when they should not be. For me it's not that easy to make a review this way.

- Cristian Oneț


On Aug. 29, 2013, 5:12 p.m., Allan Anderson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112364/
> -----------------------------------------------------------
> 
> (Updated Aug. 29, 2013, 5:12 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Description
> -------
> 
> 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.
> 
> 
> This addresses bugs 312816 and 322768.
>     http://bugs.kde.org/show_bug.cgi?id=312816
>     http://bugs.kde.org/show_bug.cgi?id=322768
> 
> 
> Diffs
> -----
> 
>   kmymoney/dialogs/investactivities.cpp 50f33ed 
>   kmymoney/dialogs/investtransactioneditor.h 3e62c2a 
>   kmymoney/dialogs/investtransactioneditor.cpp e9f87fb 
>   kmymoney/dialogs/transactioneditor.h f07dafb 
>   kmymoney/dialogs/transactioneditor.cpp 39049cf 
>   kmymoney/views/kgloballedgerview.h 04a6303 
>   kmymoney/views/kgloballedgerview.cpp 78d98b2 
>   kmymoney/widgets/register.h eebe78d 
>   kmymoney/widgets/register.cpp 1bdf5bd 
>   kmymoney/widgets/transactionform.cpp 642e98f 
> 
> Diff: http://git.reviewboard.kde.org/r/112364/diff/
> 
> 
> Testing
> -------
> 
> Extensive editing of sample files, and changing back and forth between different activity types, which tended to show
> problem areas. atype run.
> 
> 
> Thanks,
> 
> Allan Anderson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20130910/88f4b704/attachment.html>