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