Review Request: Add a confirmation window when emptying the trash
    Parker Coates 
    parker.coates at kdemail.net
       
    Mon Mar 29 20:46:02 BST 2010
    
    
  
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3434/#review4754
-----------------------------------------------------------
/trunk/KDE/kdelibs/kio/kio/jobuidelegate.cpp
<http://reviewboard.kde.org/r/3434/#comment4267>
    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.
- Parker
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
> 
>
    
    
More information about the kde-core-devel
mailing list