Skip to content
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

I can only navigate and gg G, other keys like "d" for delete are not working #13

Closed
felipejfc opened this issue Aug 31, 2016 · 5 comments
Labels

Comments

@felipejfc
Copy link

Are all keys features implemented yet?

@KenjiTakahashi
Copy link
Contributor

I'm guessing that delete does not work the way you think it does :-).
It moves marked file(s) into a "buffer", then when you do "paste", those files are moved into the destination. It does not actually erase the files from the disk.

@gokcehan perhaps some info about how this works should be added to the docs? I've had a problem with it as well.

@felipejfc
Copy link
Author

@KenjiTakahashi yes, that was the case :)

gokcehan added a commit that referenced this issue Aug 31, 2016
@gokcehan
Copy link
Owner

gokcehan commented Sep 2, 2016

@KenjiTakahashi @felipejfc I forgot to write, I have added a section to the tutorial but this is still less than ideal. A visual cue might be nice (e.g. graying out deleted files) but since other clients can change the deleted files as well, we need to check the server each time we draw. Without checking the server it might still be worthwhile though, even if it sometimes might show the wrong deleted files.

@KenjiTakahashi
Copy link
Contributor

KenjiTakahashi commented Sep 3, 2016

I think that displaying wrong can at times get just as unintuitive as not displaying anything.
Maybe have a way to display currently yanked/deleted files in a separate "buffer"? ranger does this for "work queue" [or whatever it was called, but I think it was this ;-)].
Or do some kind of push notifications, so that (other) clients get to know something was yanked/deleted without actively polling for it?

gokcehan added a commit that referenced this issue Oct 1, 2016
gokcehan added a commit that referenced this issue Nov 7, 2016
gokcehan added a commit that referenced this issue Nov 8, 2016
gokcehan added a commit that referenced this issue Nov 10, 2016
gokcehan added a commit that referenced this issue Nov 10, 2016
gokcehan added a commit that referenced this issue Nov 10, 2016
@gokcehan
Copy link
Owner

Ok so the prototype for "push notification" thing is in place. I think we can close this issue now.

@gokcehan gokcehan added the bug label Sep 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants