This adds the "Open With..." option known from Dolphin to the icon applet when it points to a file. Also adds a "Open Containing Folder".
While at it, moves the action logic for jump list actions to C++ side
Details
Details
- Jump list actions still work
- Open with works, both the primary entry, submenu, and "Open with" dialog
- "open containing folder" opens the parent folder and scrolls to the file
Diff Detail
Diff Detail
- Repository
- R120 Plasma Workspace
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
if i understood correctly the code, for deskto files that points to an app shortcut those two entries would not be added, right? (that's i think how it should work)
applets/icon/iconapplet.cpp | ||
---|---|---|
235 | Make it smart pointer or assign nullptr after deletion or you hit a crash below |
Comment Actions
for deskto files that points to an app shortcut those two entries would not be added, right?
Yes, "Open With" for apps doesn't make sense (what, open desktop file in editor?) and "Open containing folder" would open /usr/share/applications, very useful ;)
Comment Actions
- Only offer Open With when for protocols that support listing (avoids it trying to open the "parent folder" of a website url)