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

Toggle the long/short path in current location/quikfix window #98

Merged
merged 4 commits into from Jun 6, 2020

Conversation

lsaint
Copy link
Contributor

@lsaint lsaint commented May 14, 2020

2e385e6

Toggle the long/short path in current location/quikfix window.
Only available when location/quickfix is open.

Example:
nmap <F7> <Plug>(qf_shorten_path_toggle)

@romainl
Copy link
Owner

romainl commented May 14, 2020

@lsaint thank you. Give me a day or two to come back with a review.

@romainl romainl merged commit 6769786 into romainl:master Jun 6, 2020
@romainl
Copy link
Owner

romainl commented Jun 6, 2020

More like three weeks…

Thank you, I tried my hands at what I considered a simpler approach in the mean time but it kept becoming more complex for no real benefit over this.

Thank you.

romainl pushed a commit that referenced this pull request Jun 10, 2020
@jonwalstedt
Copy link

@romainl This would be a really nice feature to have, any reason to why this merge request reverted? Is there an alternative approach or something similar in the pipeline?

@romainl
Copy link
Owner

romainl commented Jul 3, 2020

@jonwalstedt I agree it would be nice but this specific implementation is flawed: it threw errors on startup when doing $ vim -q foobar, for example.

FWIW, the approach doesn't seem inherently wrong but it is not complete.

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

Successfully merging this pull request may close these issues.

None yet

3 participants