syl20bnr / evil-lisp-state

An evil state to navigate Lisp code and modify it with smartparens
GNU General Public License v3.0
91 stars 26 forks source link

Strange behaviour of insert-sexp-before for nested one-line sexps #28

Open abcdw opened 8 years ago

abcdw commented 8 years ago

Reported in spacemacs repo: https://github.com/syl20bnr/spacemacs/issues/6788

Description

It inserts new sexp inside previous symbol.

Reproduction guide

Observed behaviour:

(firs ()t
 (second))

Expected behaviour:

(first () (second))

or

(first ()
 (second))

System Info

((auto-completion :variables auto-completion-return-key-behavior 'complete auto-completion-tab-key-behavior 'complete auto-completion-enable-help-tooltip t auto-completion-complete-with-key-sequence "C-m" auto-completion-enable-snippets-in-popup t)
 (ranger :variables ranger-show-preview t)
 lua
 (c-c   :variables c-c  -enable-clang-support t)
 (clojure :variables cider-repl-history-size 1000 cider-repl-history-file "~/.cider_history")
 emacs-lisp java python ipython-notebook html latex markdown org yaml version-control git
 (evil-snipe :variables evil-snipe-enable-alternate-f-and-t-behaviors t)
 semantic gtags syntax-checking spell-checking
 (colors :variables colors-enable-rainbow-identifiers nil)
 spacemacs-layouts)
TheBB commented 8 years ago

Fixed by https://github.com/syl20bnr/evil-lisp-state/pull/29

abcdw commented 8 years ago

Pretty fast fix, thanks a lot.

abcdw commented 7 years ago

Seems not to be fixed in dev branch of spacemacs.

bitclick commented 3 years ago

i am affected by this in 2021 with spacemacs dev branch

the patch #29 works for me, when i apply it manually. why is this not merged?