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

[feature request] Add option to omit current file from MRU list #1

Closed
af opened this issue Apr 16, 2016 · 4 comments
Closed

[feature request] Add option to omit current file from MRU list #1

af opened this issue Apr 16, 2016 · 4 comments

Comments

@af
Copy link

af commented Apr 16, 2016

Hi,
What are your thoughts on adding an option to remove the current file from the MRU list? Since it's already open, I would never select it.

Thanks for a neat plugin, decent MRU support has been the one thing I've missed since switching to FZF.

@tweekmonster
Copy link
Owner

Woah, I'm not alone! There are at least 5 of us according to the traffic graph! ✊

What are your thoughts on adding an option to remove the current file from the MRU list? Since it's already open, I would never select it.

Never thought about it until you asked, but it makes sense. I can add an --exclude flag to the shell script. But, I think it could just be hard coded and not configurable on the condition that it excludes the file that's in the current buffer. I still use FZF to select already-opened files that aren't in the current buffers, or to open the same file in a split. What do you think?

@af
Copy link
Author

af commented Apr 16, 2016

I'm clearly biased, but I'd be fine with hardcoding it. I was only referring to the current buffer's file, I agree that files open in other buffers should not be excluded.

Is there a use-case for selecting the current file? I can't think of any.

@tweekmonster
Copy link
Owner

Is there a use-case for selecting the current file? I can't think of any.

I don't know. I guess maybe if you wanted to quickly see what was last opened. But, there's 5 of us, so it's not like it's going to be an earth shattering change.

@af
Copy link
Author

af commented Apr 17, 2016

Works great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants