----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122332/#review75062 -----------------------------------------------------------
src/qmlcontrols/kquickcontrolsaddons/kquickcontrolsaddonsplugin.cpp <https://git.reviewboard.kde.org/r/122332/#comment51955> older changes? I told you previously but I think you missed it, we have PlasmaPlatformComponents.IconDialog, so lets not reinvent wheel. it is already used by activity manager - Bhushan Shah On Jan. 31, 2015, 3:37 a.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122332/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2015, 3:37 a.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