----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127679/#review94760 -----------------------------------------------------------
You did not set the branch of this review request, is it master or frameworks? - Cristian Oneț On April 17, 2016, 11:14 p.m., Mitch Frazier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127679/ > ----------------------------------------------------------- > > (Updated April 17, 2016, 11:14 p.m.) > > > Review request for KMymoney. > > > Repository: kmymoney > > > Description > ------- > > KDialog does not work well with Qt Designer. > > Changing the widget class from KDialog to QDialog is generally > all that's required. Even though changing from KDialog to > QDialog does remove the OK and Cancel buttons in Designer, > these are added automatically because the dialog class itself > is still KDialog and it by default adds these buttons. > > One of the dialogs did require the adding of a setButtons() call > to the constructor to place additional buttons on the dialog > that were previously specified in the .ui file. > > Before and after screenshots show what the kfindtransactiondlgdecl.ui > file looks like in Qt Designer before with KDialog and after with QDialog. > > Note that one of the dialogs, ksplittransactiondlgdecl.ui appears to > be unused and can probably be deleted, although I did not test that. > > > Diffs > ----- > > kmymoney/dialogs/kcurrencycalculatordecl.ui 0bcfd40 > kmymoney/dialogs/kcurrencyeditdlgdecl.ui 1527455 > kmymoney/dialogs/kfindtransactiondlg.cpp 7603007 > kmymoney/dialogs/kfindtransactiondlgdecl.ui 44fa861 > kmymoney/dialogs/kmymoneypricedlgdecl.ui a484504 > kmymoney/dialogs/ksortoptiondlg.ui e7e28e5 > kmymoney/dialogs/ksplitcorrectiondlg.ui fb6d337 > kmymoney/dialogs/ksplittransactiondlgdecl.ui 6b1c706 > > Diff: https://git.reviewboard.kde.org/r/127679/diff/ > > > Testing > ------- > > Tested the affected dialogs for correct look and function. > > > File Attachments > ---------------- > > Before KDialog > > https://git.reviewboard.kde.org/media/uploaded/files/2016/04/17/84cb3eaf-fd0d-44c8-80a3-70fe3d6ba2b3__before-kdialog.png > After QDialog > > https://git.reviewboard.kde.org/media/uploaded/files/2016/04/17/eb4e6c79-c0d0-4d9b-8354-8e82f692cb90__after-qdialog.png > > > Thanks, > > Mitch Frazier > >
