----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122332/#review75197 -----------------------------------------------------------
to do modality you have to use exec i guess - Marco Martin On Jan. 30, 2015, 10:07 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122332/ > ----------------------------------------------------------- > > (Updated Jan. 30, 2015, 10:07 p.m.) > > > Review request for Plasma and Daniel Vrátil. > > > Repository: kdeclarative > > > Description > ------- > > This patch adds KQuickControls wrapper around KIconDialog similar to how the > ColorDialog and other QtQuick Dialogs work. This can be used, for instance, > in Kickoff's config UI to provide a picker for a custom item. > > It is an initial draft and lacks for example window modality as I couldn't > figure out how QtQuick Dialogs do that (some PlatformDialogHelper magic > inside) and I'm also not sure about the lifecycle/ownership of the dialog, > I've seen a lot of fixes for issues in that area on Review Board. > > > Diffs > ----- > > src/qmlcontrols/kquickcontrolsaddons/CMakeLists.txt 65e28ff > src/qmlcontrols/kquickcontrolsaddons/icondialog.h PRE-CREATION > src/qmlcontrols/kquickcontrolsaddons/icondialog.cpp PRE-CREATION > src/qmlcontrols/kquickcontrolsaddons/kquickcontrolsaddonsplugin.cpp 289f1ed > > Diff: https://git.reviewboard.kde.org/r/122332/diff/ > > > Testing > ------- > > For testing I added a button to Kickoff that allows to open the dialog and > the button icon source is bound to the dialog's iconName property. Didn't > test the icon source/user/custom path stuff. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
