----------------------------------------------------------- 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 > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel