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

Allow slashes to cd relative to warp point #66

Open
towc opened this issue Mar 6, 2019 · 5 comments · May be fixed by #97
Open

Allow slashes to cd relative to warp point #66

towc opened this issue Mar 6, 2019 · 5 comments · May be fixed by #97
Assignees

Comments

@towc
Copy link
Contributor

towc commented Mar 6, 2019

Say this is my setup:

cd some/path
mkdir y
wd add x
cd ~

Something I keep accidentally doing is this:

wd x/y

Instead, I guess the intended workflow is

wd x
cd y

Any chance this could be a feature? Maybe that would need enabling in the rc file, so it doesn't conflict with people's existing warp points?

Would be lovely if you could also wd x/y/z/other, not just one level deep

@towc
Copy link
Contributor Author

towc commented Mar 6, 2019

oh, just noticed #17. Maybe this should be documented somewhere more visible?

So I guess the intended flow is wd x y. Still, can this be made to work with slashes as well?

@mfaerevaag
Copy link
Owner

mfaerevaag commented Mar 6, 2019

Ah you're right. We have forgot to document it actually.

And yes, I do believe it's possible, but it should considered whether it could lead to ambiguities when having warp names with slashes. That being said, I do remember we looked into this, but found that it wasn't trivial to solve due to the zsh completion syntax. You're most welcome to give it a shot yourself, though. If you submit a PR we'll be happy to look at it 😄

@alpha-tango-kilo alpha-tango-kilo self-assigned this Sep 4, 2020
@alpha-tango-kilo alpha-tango-kilo changed the title allow slashes to cd relative to warp point Allow slashes to cd relative to warp point Sep 4, 2020
alpha-tango-kilo added a commit to alpha-tango-kilo/wd that referenced this issue Sep 4, 2020
@alpha-tango-kilo alpha-tango-kilo linked a pull request Sep 4, 2020 that will close this issue
@alpha-tango-kilo
Copy link
Collaborator

This is now a thing! Work in progress, but the core of the functionality is there 😄

See the PR above

@alpha-tango-kilo alpha-tango-kilo added this to the v0.6 milestone Sep 4, 2020
@p1r473
Copy link
Contributor

p1r473 commented Jun 12, 2024

@alpha-tango-kilo need any help with this one? Is this stuck?

@alpha-tango-kilo
Copy link
Collaborator

See the comments I left on the PR

@alpha-tango-kilo alpha-tango-kilo removed this from the v0.6 milestone Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants