-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125991/#review88427
-----------------------------------------------------------


Note to self: Disable drag area when applet is pending uninstall

- Kai Uwe Broulik


On Nov. 8, 2015, 12:23 vorm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125991/
> -----------------------------------------------------------
> 
> (Updated Nov. 8, 2015, 12:23 vorm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> The no longer enabled tooltip also contained the uninstall button. This 
> restores the uninstall functionality similar to the wallpaper dialog.
> 
> User-installed applets get an uninstall button. I chose to always show it 
> rather than on hover since only a few applets are likely to be uninstallable 
> and I don't want to play the guessing-game, perhaps we should introduce a 
> filter for "User-installed applets" similar to the (less useful) "running" 
> ones. Clicking the button will queue the uninstallation which can be 
> cancelled anytime the dialog is still opened. As soon as the dialog closes, 
> the actual uninstall is executed.
> 
> Multiple applets can be enqueued simultaneously.
> 
> 
> Diffs
> -----
> 
>   desktoppackage/contents/explorer/AppletDelegate.qml 5a81b91 
>   desktoppackage/contents/explorer/WidgetExplorer.qml af27229 
> 
> Diff: https://git.reviewboard.kde.org/r/125991/diff/
> 
> 
> Testing
> -------
> 
> Installed plasmoid through drag and drop, uninstalled after fixing an 
> uninstall bug (Review 125990)
> 
> The tooltip says "Undo uninstall". I just noticed I should probably fade the 
> applet name too.
> 
> 
> File Attachments
> ----------------
> 
> Uninstall before
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/11/08/adcb91e0-a8c2-4481-89f1-139b4e716da3__uninstall1.png
> Undo uninstall
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/11/08/ebdae6ad-28bd-42f0-bb36-bfa6dfd3cc30__uninstall2.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to