Review Request: Add a confirmation window when emptying the trash

[Nicolas Lécureuil]

> On 2010-03-29 19:46:06, Parker Coates wrote:
> > /trunk/KDE/kdelibs/kio/kio/jobuidelegate.cpp, lines 107-116
> > <http://reviewboard.kde.org/r/3434/diff/1/?file=21830#file21830line107>
> >
> >     I'm not sure that I understand how this code works. You don't seem to be adding a new "don't ask again" key.
> >     
> >     Is the intention that the "don't ask again" key of "Move to Trash" dialog is being reused?
> >     
> >     I don't think that's a very good idea.
> 
> Nicolas Lécureuil wrote:
>     ah yes there is a pb here, this should use the other one.
>     thanks i look this pb
> 
> Parker Coates wrote:
>     That's not actually what I was trying to say. I think the "Empty Trash" dialog should have its own key. As I user, I wouldn't expect checking the box in the "Empty Trash" dialog to have anything to do with the box in the "Delete Files" dialog. The fact that they are closely related internally is just an the implementation detail. To the user, they seem like separate actions.

This is what i wanted to tell and what i am doing :)


- Nicolas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3434/#review4754
-----------------------------------------------------------


On 2010-03-29 14:46:16, Nicolas Lécureuil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3434/
> -----------------------------------------------------------
> 
> (Updated 2010-03-29 14:46:16)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Summary
> -------
> 
> When the user empty the trash ( when using trash:/ in a desktop file ) there is no confirmation window.
> This patch add a confirmation window.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/apps/lib/konq/konq_operations.cpp 1108647 
>   /trunk/KDE/kdelibs/kio/kio/jobuidelegate.h 1108647 
>   /trunk/KDE/kdelibs/kio/kio/jobuidelegate.cpp 1108647 
> 
> Diff: http://reviewboard.kde.org/r/3434/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nicolas
> 
>