-
-
Notifications
You must be signed in to change notification settings - Fork 205
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option to delete selected files #38
Comments
Sure, I'll add those. |
Thanks, maybe other options like "copy to" and "move to" and select any folder where to put such selected files would be a great addition? |
@spsf64 I just added the option to delete files and move them to trash. Currently the deleted files will still show up in the results (however the icon will indicate that they have been removed). But this will be fixed when I add file system monitoring - which will pick up delete or move (to trash) events automatically. |
Wow, great. :-)
Can this also delete files outside the home directory?
…On Fri, Dec 2, 2016 at 7:22 PM, Christian Boxdörfer < ***@***.***> wrote:
@spsf64 <https://github.com/spsf64> I just added the option to delete
files and move them to trash. Currently the deleted files will still show
up in the results (however the icon will indicate that they have been
removed). But this will be fixed when I add file system monitoring.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#38 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKbw51m216ap6ImBfMrGQld3paPd72tks5rEGH-gaJpZM4KrJWX>
.
|
@orschiro, that depends on the permissions of the parent directory. If you have permission to modify it (write permission) you can delete files in there. So in general you can't delete every file because usually your user can't modify everything. However I could use PolicyKit for example, then you get prompted for a password when you want to modify system files. I'll have a look how this can be done and maybe add an option. |
@cboxdoerfer thanks! I will check this later today. |
@spsf64, there's some progress on a prototype but it's nothing usable yet and I'm still not sure about some of the details. Maybe by the end of this month I'll be able to publish some of the progress. The reason why this takes so long is because I first need to implement a byte code compiler and interpreter for this to work nicely and efficiently. |
@cboxdoerfer Just checked and the option works perfectly! |
@spsf64, yes sure. Fixed it. |
Wow, this one was fassst! Thank you, have a nice weekend. |
@spsf64, np, you too. :) |
Maybe even 2 options like "delete" (permanently) and "move to trash"
Thanks for this great project!
The text was updated successfully, but these errors were encountered: