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

Support partial-completion for files #390

Merged

Conversation

clemera
Copy link
Collaborator

@clemera clemera commented Jan 18, 2021

See #385, possible partial completion candidates are shown when the directory of the input doesn't exist. This makes it fit nicely into the way Selectrum handles file completions, the query doesn't needs to be done later when refining, the querying and returning of possible candidates are all handled by the dynamic completion function.

@clemera clemera force-pushed the support-partial-completion-for-files branch 2 times, most recently from 97025d8 to 9d95160 Compare January 18, 2021 16:21
@clemera clemera force-pushed the support-partial-completion-for-files branch from 9d95160 to e39d827 Compare January 18, 2021 16:28
@clemera clemera force-pushed the support-partial-completion-for-files branch from 21a4cb7 to ac53066 Compare January 18, 2021 16:48
@clemera clemera force-pushed the support-partial-completion-for-files branch from c3dd756 to bee6873 Compare January 18, 2021 16:52
@clemera clemera force-pushed the support-partial-completion-for-files branch from e99a07c to a944d64 Compare January 18, 2021 19:03
@clemera clemera force-pushed the support-partial-completion-for-files branch from a944d64 to 7181cb9 Compare January 18, 2021 21:29
@clemera clemera force-pushed the support-partial-completion-for-files branch from 07d3724 to 21ab656 Compare January 18, 2021 21:30
(is-tramp-path
;; Check for tramp path, see
;; `tramp-initial-file-name-regexp'.
(string-match-p "\\`/[^/:]+:[^/:]*:" path))
Copy link
Contributor

Choose a reason for hiding this comment

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

FWIW, there is also file-remote-p

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, I thought this would only catch real remote paths and not /sudo:: for example I will switch to that.

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

Successfully merging this pull request may close these issues.

2 participants