----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121747/#review72757 -----------------------------------------------------------
Ship it! autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50673> Coding style. autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50674> Coding style. autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50675> Coding style. autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50677> Coding style. autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50678> One more sace. autotests/kfiledialog_unittest.cpp <https://git.reviewboard.kde.org/r/121747/#comment50680> Coding style. Looks like i owe you a big sorry. Much (or all?) of those coding style issues seem to introduced by... me... Anyhow, looks OK to me. - Mark Gaiser On dec 29, 2014, 10:38 p.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121747/ > ----------------------------------------------------------- > > (Updated dec 29, 2014, 10:38 p.m.) > > > Review request for KDE Frameworks, Lukáš Tinkl, Mark Gaiser, and Martin > Klapetek. > > > Repository: frameworkintegration > > > Description > ------- > > Commit 0b8ab82e7fa9ea0948a841ebf8523b62da made the view mode a choice of > the user (as it always was in KDE code) rather than a choice of the > developer (as QFileDialog::setViewMode allows). > > Typical Qt/KDE difference: Qt empowers the developer, KDE empowers the user. > I agree with Lukas that the latter makes more sense in KDE code. > > => I rewrote the unittest to check that the user's changes are restored > on the next file dialog. > > > Diffs > ----- > > autotests/kfiledialog_unittest.cpp e5108ead5a381864cfa2b8ddf62b5d6c64b13db9 > > Diff: https://git.reviewboard.kde.org/r/121747/diff/ > > > Testing > ------- > > > Thanks, > > David Faure > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
