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

* layers/+spacemacs/spacemacs-defaults/funcs.el: Enhance copying file path for dired-mode #16766

Merged
merged 2 commits into from
Jan 1, 2025

Conversation

sunlin7
Copy link
Contributor

@sunlin7 sunlin7 commented Dec 28, 2024

Currently pressing SPC f y y on a dired-mode buffer will get an error.

This PR enhances the users experience by support copying file path on a dired-mode buffer (SPC f y y in a dired buffer).

layers/+spacemacs/spacemacs-defaults/funcs.el Outdated Show resolved Hide resolved
layers/+spacemacs/spacemacs-defaults/funcs.el Outdated Show resolved Hide resolved
layers/+spacemacs/spacemacs-defaults/funcs.el Outdated Show resolved Hide resolved
@sunlin7 sunlin7 force-pushed the copy-file-path-enhance branch from eb44da2 to bba77f4 Compare December 29, 2024 05:43
@sunlin7
Copy link
Contributor Author

sunlin7 commented Dec 29, 2024

@bcc32 Thanks for the comments, new changes pushed, please help review again.

@bcc32 bcc32 self-requested a review January 1, 2025 00:03
@bcc32
Copy link
Collaborator

bcc32 commented Jan 1, 2025

Pushed some small tweaks, to (1) avoid raising a user-error and then immediately catching it and (2) use logic consistent with SPC f y d to determine what directory path to copy. Happy to merge once confirmed these work for you.

@sunlin7
Copy link
Contributor Author

sunlin7 commented Jan 1, 2025

@bcc32 Thank you for the improvement, and I had tried the last one on my local, they work perfect ! Thanks

@bcc32 bcc32 merged commit 0d176e2 into syl20bnr:develop Jan 1, 2025
7 of 9 checks passed
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.

2 participants