Review Request 123211: Ask for confirmation before deleting a wallpaper.
Thomas Pfeiffer
colomar at autistici.org
Wed Apr 1 16:19:01 UTC 2015
> On April 1, 2015, 3:58 nachm., Kai Uwe Broulik wrote:
> > I don't think the wallpaper dialog should ever delete an actual image file, especially not when it's in the user's Pictures folder. Only exception is when it has been installed through GHNS in which case it is in some hidden dot folder.
> >
> > As for the confirmation, I think we should rather provide an undo functionality so you don't accidentally remove the wallpaper from the list and then have to search it on the filesystem again.
>
> Marco Martin wrote:
> I think the patch for now is fine as is, modulo using QtQuickcontrols.
> the file is deleted in ghns case and just the entry removed in case of photo on the hard drive.
> the wording on the dialog should reflect that.
The patch may be fine from a technical perspective, but it has a considerable negative impact on user experience, and therefore should not be shipped anyway.
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123211/#review78350
-----------------------------------------------------------
On April 1, 2015, 3:27 nachm., Antonis Tsiapaliokas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123211/
> -----------------------------------------------------------
>
> (Updated April 1, 2015, 3:27 nachm.)
>
>
> Review request for Plasma.
>
>
> Bugs: 338729
> https://bugs.kde.org/show_bug.cgi?id=338729
>
>
> Repository: plasma-workspace
>
>
> Description
> -------
>
> This patch is adding a confirmation dialog which is being called before we remove a wallpaper.
>
>
> Diffs
> -----
>
> wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml aee2d3f
> wallpapers/image/imagepackage/contents/ui/config.qml 2108082
>
> Diff: https://git.reviewboard.kde.org/r/123211/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> dialog
> https://git.reviewboard.kde.org/media/uploaded/files/2015/04/01/5bfd2d7c-8baa-4b80-ad20-0844aafdb3a9__deletion_dialog.png
>
>
> Thanks,
>
> Antonis Tsiapaliokas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150401/414bc332/attachment.html>
More information about the Plasma-devel
mailing list