Also fix a logic error while at it.
Details
Details
- Reviewers
hein - Group Reviewers
Plasma - Commits
- R120:eff4bc7c3cf3: [LauncherTasksModel] Support applications: URLs in requestOpenUrls
I noticed that it would crash when I dropped a file onto my Kate launcher. Then I found the logic bug, and then I found that the entire thing actually didn't work because we forgot applications: special URL handling
I can now drop a file onto my Kate launcher again and it opens it in Kate. When app is running dropping still works. Multiple files also work.
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
I can't help but feel increasingly bad about the whole applications: URLs thing. If we have to patch 5000 places something is not right. We need to come up with a better way some time soon. What are we doing wrong? Maybe we should just pass KServices around? Hmm.
In the meantime, this patch fixes a bug, so +1.