> On Juni 17, 2015, 7:36 nachm., Sebastian Kügler wrote: > > applets/trash/plugin/trashplugin.cpp, line 30 > > <https://git.reviewboard.kde.org/r/124112/diff/2/?file=380371#file380371line30> > > > > I think we still want to delete this one *somewhere*?
>From the documentation: NOTE: A QObject singleton type instance returned from a singleton type provider is owned by the QML engine. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124112/#review81537 ----------------------------------------------------------- On Juni 17, 2015, 4:24 nachm., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124112/ > ----------------------------------------------------------- > > (Updated Juni 17, 2015, 4:24 nachm.) > > > Review request for Plasma. > > > Bugs: 348378 > https://bugs.kde.org/show_bug.cgi?id=348378 > > > Repository: plasma-desktop > > > Description > ------- > > This allows to drag files onto the trash applet to trash them. > Also, the code is cleaned up a bit and it is given a sensible minimum size > (so the label always fits) as well as the query dialog has been replaced by > the standard KIO dialog. > > > Diffs > ----- > > applets/trash/package/contents/ui/main.qml 7715439 > applets/trash/plugin/CMakeLists.txt 9012e66 > applets/trash/plugin/trash.h PRE-CREATION > applets/trash/plugin/trash.cpp PRE-CREATION > applets/trash/plugin/trashplugin.cpp 7e866a9 > > Diff: https://git.reviewboard.kde.org/r/124112/diff/ > > > Testing > ------- > > Trashed one or multiple files on both desktop and panel. > When I dismiss the message box, however, Plasma will create an applet for the > file dragged. How can I tell it I accepted the drop? Also, it crashes in > Client::~Client() () from /usr/lib/x86_64-linux-gnu/libQPulseAudioPrivate.so > at times after dismissing the dialog. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
