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

Add helm-find binding #3114

Closed
wants to merge 2 commits into from
Closed

Add helm-find binding #3114

wants to merge 2 commits into from

Conversation

justbur
Copy link
Contributor

@justbur justbur commented Sep 23, 2015

My proposal for #3108

Replaces SPC f l for find-file-literally. If that's a useful binding then maybe we can move this. I like fl for find "locally" and fL for find "globally" on the system.

@robbyoconnor
Copy link
Contributor

👍

@bmag
Copy link
Collaborator

bmag commented Sep 26, 2015

👍
Do you think adding documentation about installing find (and locate, while we're at it) on Windows is necessary? I don't use Emacs on Windows, so I don't know if it's really needed.

@justbur
Copy link
Contributor Author

justbur commented Sep 26, 2015

Yes that would be nice. I can put it together

@justbur
Copy link
Contributor Author

justbur commented Sep 28, 2015

Added note to windows users

@sooheon
Copy link

sooheon commented Sep 28, 2015

I was the one who suggested find literally, it's extremely useful to get around emacs' slow long line rendering for really big files, but it is niche enough to move I guess. "f C-l" or "F l" come to mind, any better options?

@justbur
Copy link
Contributor Author

justbur commented Sep 28, 2015

@sooheon This is a different approach, but wouldn't it makes sense to add a helm action for helm-find-files that allows you to open it using find-file-literally?

@sooheon
Copy link

sooheon commented Sep 28, 2015

@justbur that way does seem more elegant :)

@justbur
Copy link
Contributor Author

justbur commented Sep 28, 2015

@sooheon try this

(helm-add-action-to-source "Find File Literally" 'find-file-literally helm-source-find-files 2)

@sooheon
Copy link

sooheon commented Sep 28, 2015

@justbur yep thanks. And if anyone reading uses swiper/ivy, the following is the equivalent:

(ivy-set-actions
     'counsel-find-file
     '(("l" find-file-literally "Find file literally")))

Replaces `SPC f l` for `find-file-literally`

Also, add note for windows users about find and locate
@justbur
Copy link
Contributor Author

justbur commented Sep 28, 2015

Great. Added the helm action to this pr

To make up for removing it from the `SPC f l` binding in the previous
commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants