-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
fd
, newline, smart parens and python mode
#5261
Comments
Confirmed with my different escape sequence ( I don't know if it can be avoided. This effect is due to this:
is the correct indentation that python-mode gives to this line if there is only the closing parenthesis. If there is something on the second line, like:
the indentation is then shifted by 4 spaces, which is also the indentation the python-mode gives. I suppose whenever people press This should probably be reported upstream to |
Yeah this really does kill kittens. Thanks for the tip @StreakyCobra. I've had this issue in Python for a while and I never connected it to evil-escape. |
For now I will be using (add-hook 'evil-insert-state-exit-hook 'indent-according-to-mode) |
Maybe using undo instead of the removing the char may work in evil-escape. I'll test it. |
Just a note (from #5313) This happens in ruby-mode as well. Test text (pipe is cursor position): def test
end| |
I retract my workaround. It's maddening. |
This has been reported upstream: syl20bnr/evil-escape#69 We can probably close this and follow the progress there. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid! |
Emacs: 24.5.1 (Windows).
Spacemacs: latest
develop
.How to reproduce: in Python-mode type
( RET f d
which will result ininstead of
because
f
seems to shift the second parenthesis in an irreversible fashion (closing the parentheses manually works ok of course).This is a minor issue, not sure if it can be considered a bug but still, reporting it for the record.
The text was updated successfully, but these errors were encountered: