----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124472/#review82986 -----------------------------------------------------------
applets/icon/plugin/icon_p.cpp (line 102) <https://git.reviewboard.kde.org/r/124472/#comment57301> IMO it would make sense to open dropped urls only if it is supported by application.. like only accept text files on kate and not image files. - Bhushan Shah On July 26, 2015, 4:10 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124472/ > ----------------------------------------------------------- > > (Updated July 26, 2015, 4:10 p.m.) > > > Review request for Plasma and Bhushan Shah. > > > Repository: plasma-workspace > > > Description > ------- > > Brings back the ability to drop files onto a icon widget and have the > associated application open it. Dropping something onto a folder does not yet > work because it should use the konq menu thingie (Copy/Move/Link) but I don't > want yet another copy of that (there's one in the folderview containment). > > It should probably check already on drag enter whether it's a viable drop > target (eg. executable or folder), also do some coding style cleanups I > thought I had already committed earlier. > > > Diffs > ----- > > applets/icon/package/contents/ui/main.qml e0e1df7 > applets/icon/plugin/icon_p.h ba2d9f7 > applets/icon/plugin/icon_p.cpp 18806fc > > Diff: https://git.reviewboard.kde.org/r/124472/diff/ > > > Testing > ------- > > Dragged Gwenview onto the desktop, an icon was created. > Dropped one picture onto it, Gwenview opened it. > Dropped two pictures onto, Gwenview showed both. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
