----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122893/#review77409 -----------------------------------------------------------
Ship it! Hopefully I did not miss anything this time :) - Christoph Feck On March 13, 2015, 6:36 a.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122893/ > ----------------------------------------------------------- > > (Updated March 13, 2015, 6:36 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kiconthemes > > > Description > ------- > > Populate the dialog on showEvent, this way you can use the regular QDialog > show() and exec() methods and use, for instance, Qt::WindowModal rather than > having showDialog() or openDialog() mess with modality. showDialog() could > probably be deprecated then. > > I don't know whether this is binary compatible. "you can safely reimplement a > virtual function defined in one of the base classes only if it is safe that > the programs linked with the prior version call the implementation in the > base class rather than the derived one" > > > Diffs > ----- > > src/kicondialog.h 83572a0 > src/kicondialog.cpp 75f9f01 > > Diff: https://git.reviewboard.kde.org/r/122893/diff/ > > > Testing > ------- > > Just calling show() on the KIconDialog works as expected, the icon button in > kfileproperties thingie dialog still works. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
