D29082: WIP Dolphin Service: Show error message when uninstalling packages

Alexander Lohnau noreply at phabricator.kde.org
Wed Apr 22 07:49:53 BST 2020


alex added inline comments.

INLINE COMMENTS

> broulik wrote in servicemenuinstaller.cpp:276
> Why not show a `KMessageBox`?
> Alternatively, offer the user to do that?
> "This thing needs to be installed [Open in whatever app for the mime type, e.g. Discover] [Cancel]"

So far the messages have been show with kdialog, thats why I didn't wanted to mix that too much up in this patch.

But there is an issue when opening the files for uninstalling by the mime type:
If you install a local deb file in discover you get the uninstall button afterwards.
But if you close discover and launch the local deb file again you only get the install option.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D29082

To: alex, #dolphin, ngraham, nicolasfella, elvisangelaccio
Cc: broulik, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200422/393deeef/attachment.htm>


More information about the kfm-devel mailing list