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

enhance the Invoke-FzfPsReadlineHandlerProvider #198

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

metasong
Copy link

@metasong metasong commented Nov 8, 2022

copy c:\programs\(cursor here) d:\apps press ctrl-t to trigger function Invoke-FzfPsReadlineHandlerProvider, then select some path will expand

`copy c:\programs\(cursor here) d:\apps`  press ctrl-t to trigger function Invoke-FzfPsReadlineHandlerProvider, then select some path will expand
@kelleyma49
Copy link
Owner

I think I see what you're trying to do, but unfortunately you're code breaks other instances. See example below.

CtrlTDoesNotWork

Copy link
Owner

@kelleyma49 kelleyma49 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks the normal case of CTRL+T.

@Konstantin-Glukhov
Copy link

Konstantin-Glukhov commented Feb 8, 2023

This is a valid enhancement request.
See my pool request #215 to address the issue.
It works fine for both current folder and supplied path.

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.

3 participants