Comment 55 for bug 120672

Revision history for this message
In , Mats Palmgren (matspal) wrote :

(In reply to comment #54)
> I'd question the need for a confirmation dialog.

The operation cannot be undone.

> Do we have one for Empty Trash?

No, but we probably should have one since it also cannot be undone.
I wrote confirmToProceed() with this in mind, so you could use it
as confirmToProceed('emptyTrash')... ;-)

> I'm also unclear on the need to handle
> sub-folders although I guess it doesn't hurt, other than a bit of code
> bloat...do users really have sub-folders of Junk?

"Empty Trash" removes subfolders so I think we should do it for symmetry.