WIP: Add KFileItemActionsPlugin
AbandonedPublic

Authored by broulik on May 15 2018, 1:36 PM.

Details

Reviewers
apol
Group Reviewers
Plasma
VDG
Summary

Allows to share files from Dolphin's file context menu and notification thumbnails.

Test Plan

Menu shows different actions depending on file type, e.g. imgur is only offered for images

TODO: When I click Imgur, the progress dialog shows up but the finished handler is never invoked :(

Diff Detail

Repository
R495 Purpose Library
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.May 15 2018, 1:36 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptMay 15 2018, 1:36 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.May 15 2018, 1:36 PM

Fantastic! I've wanted this for so long!

That "send as email attachment" item you have in your Dolphin context menu... is that a separate service item provided by KMail that will become obsolete with this new share menu? Seems kinda redundant to have them both...

elvisangelaccio added inline comments.
src/fileitemactionplugin/purposefileitemaction.desktop
1–8

Why not just starting with JSON metadata? (since this is a new plugin)

nicolasfella added inline comments.
src/fileitemactionplugin/sharefileitemaction.cpp
83

mimeType() can be empty which can cause problems (see D12728). Not sure if it will here though

91

You can use m_purposeMenu->menuAction()

broulik abandoned this revision.Jun 1 2018, 10:43 AM

Superseded by D13124