> On Nov. 19, 2012, 8:34 a.m., Cristian Oneț wrote: > > kmymoney/widgets/transaction.cpp, line 336 > > <http://git.reviewboard.kde.org/r/107373/diff/2/?file=95304#file95304line336> > > > > If you set the ctx.palette to option palette the selected test is not > > needed anymore.
ctx.palette = option.palette; is already present at row 333, if I remove this test the text in the selected row is black and not white... > On Nov. 19, 2012, 8:34 a.m., Cristian Oneț wrote: > > kmymoney/widgets/transaction.cpp, line 337 > > <http://git.reviewboard.kde.org/r/107373/diff/2/?file=95304#file95304line337> > > > > I would go with the following code here: > > > > ctx.palette = option.palette; > > > > Just to make sure that we use all the stuff that is set in the palette > > before calling this function. > > Already there, at row 333 - Alessandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107373/#review22206 ----------------------------------------------------------- On Nov. 19, 2012, 12:44 a.m., Alessandro Russo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107373/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2012, 12:44 a.m.) > > > Review request for KMymoney. > > > Description > ------- > > Fixed: > - when closing the split dialog focus go to tag field and not to memo field > - if the user write in the tag line edit the name of a closed tag or of a > tag already included in the transaction an information modal dialog will > appear and the line edit will be cleared. > - restore right color for text in the ledger > > > Diffs > ----- > > kmymoney/dialogs/transactioneditor.cpp 1a5e773 > kmymoney/widgets/kmymoneymvccombo.h ad6ae92 > kmymoney/widgets/kmymoneymvccombo.cpp 1053a1a > kmymoney/widgets/transaction.cpp 7525851 > > Diff: http://git.reviewboard.kde.org/r/107373/diff/ > > > Testing > ------- > > Compile and run as intended. > > > Thanks, > > Alessandro Russo > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel
