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

Define list thing to improve navigation in Emacs 31 #65

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

rrudakov
Copy link
Contributor

@rrudakov rrudakov commented Mar 7, 2025

If list thing is defined, Emacs 31 will set a few options automatically to improve navigation by lists/sexp etc.

Big thanks for the patch to Juri Linkov [email protected].


Before submitting a PR mark the checkboxes for the items you've done (if you
think a checkbox does not apply, then leave it unchecked):

  • You've run M-x checkdoc and fixed any warnings in the code you've written.
  • You've updated the changelog (if adding/changing user-visible functionality).
  • You've updated the readme (if adding/changing user-visible functionality).

Thanks!

@rrudakov rrudakov changed the title [#62] Define list thing to improve navigation in Emacs 31 Define list thing to improve navigation in Emacs 31 Mar 7, 2025
@bbatsov
Copy link
Member

bbatsov commented Mar 7, 2025

Let's also add a Caveats section in the README where we can gather up items like these, so people are not surprised by differences in the behavior on different versions of Emacs.

@bbatsov
Copy link
Member

bbatsov commented Mar 7, 2025

(I added the section, so you can just an item there)

@rrudakov
Copy link
Contributor Author

rrudakov commented Mar 7, 2025

We can also set forward-sexp-function to nil if Emacs version is < 31 to workaround the bug #62

@rrudakov rrudakov force-pushed the feature/62-list-thing branch from 6f03a07 to b51cb0d Compare March 7, 2025 16:03
If list thing is defined, Emacs 31 will set a few options automatically to
improve navigation by lists/sexp etc.

Big thanks for the patch to Juri Linkov <[email protected]>.
@rrudakov rrudakov force-pushed the feature/62-list-thing branch from b51cb0d to a82fffe Compare March 7, 2025 16:10
@rrudakov
Copy link
Contributor Author

rrudakov commented Mar 7, 2025

Let's also add a Caveats section in the README where we can gather up items like these, so people are not surprised by differences in the behavior on different versions of Emacs.

Done.

@bbatsov bbatsov merged commit 51ee8a4 into clojure-emacs:main Mar 7, 2025
3 checks passed
@bbatsov
Copy link
Member

bbatsov commented Mar 7, 2025

Thanks!

@rrudakov rrudakov deleted the feature/62-list-thing branch March 7, 2025 19:57
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