Skip to content

Fix overzealous cd tab completion#4649

Closed
mtbrock wants to merge 1 commit into
fish-shell:masterfrom
mtbrock:fix-cd-tab-completion
Closed

Fix overzealous cd tab completion#4649
mtbrock wants to merge 1 commit into
fish-shell:masterfrom
mtbrock:fix-cd-tab-completion

Conversation

@mtbrock
Copy link
Copy Markdown
Contributor

@mtbrock mtbrock commented Jan 8, 2018

Description

Changed cd completion to differentiate between cd autosuggest and cd tab
completion. cd autosuggest will find deepest unique hierarchy and cd tab
completion will not.

Fixes issue #4402

TODOs:

  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.md

Changed cd completion to differentiate between cd autosuggest and cd tab
completion. cd autosuggest will find deepest unique hierarchy and cd tab
completion will not.

Issue fish-shell#4402
@ridiculousfish
Copy link
Copy Markdown
Member

Looks really good! I'll test and then merge if it checks out. Thanks for doing this!

@ridiculousfish
Copy link
Copy Markdown
Member

Merged as bf63e06, thanks!

ridiculousfish added a commit that referenced this pull request Jan 9, 2018
@mtbrock
Copy link
Copy Markdown
Contributor Author

mtbrock commented Jan 9, 2018

Glad I could help

@zanchey zanchey added this to the fish-3.0 milestone Jan 9, 2018
@Wazzaps
Copy link
Copy Markdown

Wazzaps commented Dec 28, 2018

Actually I loved this feature. Is it possible to have this as a feature flag?

@ridiculousfish
Copy link
Copy Markdown
Member

Not suitable for a feature flag, but this might be writable as a custom completion if someone wants to try.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants