qView icon indicating copy to clipboard operation
qView copied to clipboard

Delete without trash

Open notramo opened this issue 3 years ago • 8 comments

On Linux with BTRFS, NILFS2, and ZFS, it's not necessary to have a thrash, since these filesystems support snapshots. On CLI systems which rm without thrash, it's unnecessary complication to have a thrash just for qView, and clean it periodically. qView should have an option to disable trash (enable direct deletion) in the settings.

notramo avatar Apr 27 '22 21:04 notramo

I am unable to delete files from an SMB share with no trash. This might be the reason. Instead of throwing an error, we need to issue a regular filesystem delete on SMB shares.

A9G-Data-Droid avatar Jul 15 '22 04:07 A9G-Data-Droid

I am unable to delete files from an SMB share with no trash.

I am also unable to delete files from an SMB share, but it does have a trash, at least on the server side (It's just another Mac).

Anyway, if the inability to move files to the trash on SMB shares is what's preventing deletion, then I agree with @A9G-Data-Droid, as long as there a special confirmation dialogue. I can have a checkbox for "don't ask me again".

vassudanagunta avatar Jul 16 '22 03:07 vassudanagunta

I can't believe this isn't in qView currently. If someone wants to implement this, just make a new action called "force delete" or something, absent from the context menu but present in the menubar, with a default keyboard shortcut of cmd/ctrl+del and for macOS cmd+option+backspace, or something along those lines. maybe you can add a modifier key to the QKeySequence::delete or something also.

jurplel avatar Aug 08 '23 16:08 jurplel

I fully support this improvement. So deleting files on network drives would not be a problem.

simonbcn avatar Jan 15 '24 13:01 simonbcn

We have a pull request that fixes this, #639, that's 18 months old. @jurplel not complaining, as i know you're busy and doing this for free. but i do think it impacts a lot of people, as new issues about this keep getting created.

vassudanagunta avatar Apr 29 '24 21:04 vassudanagunta

We have a pull request that fixes this, #639, that's 18 months old. @jurplel not complaining, as i know you're busy and doing this for free. but i do think it impacts a lot of people, as new issues about this keep getting created.

Yes, I absolutely need to merge it. I'm going to try to update qView again in a couple of weeks, but I am trying to coordinate a move during finals week right now.

jurplel avatar Apr 29 '24 21:04 jurplel

good luck on your finals!

vassudanagunta avatar Apr 29 '24 21:04 vassudanagunta

oh cool i didn't see that PR, going to roll my own, thanks. and good luck on your finals!

illnesse avatar May 03 '24 15:05 illnesse

Thanks. Will there be a release soon with this and other corrections?

simonbcn avatar May 07 '24 12:05 simonbcn

Gonna try to make a release soon, yes.

jurplel avatar May 07 '24 20:05 jurplel