install kipi-plugins in gwenview using appstream:/ URLs

Review Request #129072 - Created Sept. 29, 2016 and submitted

Information
Jonathan Riddell
gwenview
Reviewers
gwenview, kipi-plugins
apol, sitter

Alternative to patch https://git.reviewboard.kde.org/r/129004 for comparison, this uses an appstream:// URL to launch Plasma Discover (or whatever you have that handles those URLs) to install kipi-plugins. A file watcher will update the menu when it spots the plugins have been installed.

As discussed on other review request, I prefer the built in approach as user experience. Discover is quite a complex window that gets shown to the user and it may not be clear that the user needs to click install and wait then close Discover and check the menu again. The simple modal dialog of the other patch seems easier to users.

built it, tried to use discover to install kipi (reverted to using apt as it doesn't work on Neon currently) and noted the menu gets filled

Issues

  • 0
  • 21
  • 3
  • 24
Description From Last Updated
Jonathan Riddell
Aleix Pol Gonzalez
Jonathan Riddell
Harald Sitter
Jonathan Riddell
Aleix Pol Gonzalez
Harald Sitter
Jonathan Riddell
Harald Sitter
Jonathan Riddell
Jonathan Riddell
Harald Sitter
Jonathan Riddell
Harald Sitter
Jonathan Riddell
Jonathan Riddell
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit 77ec6c86210b7accea01ca1d2fbd15d1e64fba3c by Jonathan Riddell to branch master.
Harald Sitter

So where's the kipi RR?

  1. remains work for future Agile development sprint cycles

  2. well. good to know how much you value input.

  3. oh Harald, I love you input and did many revisions as a result of it. But at some point it makes sense to finish it off in a working state and leave further improvements for future work, we can't all be as perfect as you

Loading...