-
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
Python inferior buffer improvement #598
Conversation
Projectile takes time to load (around 0.2 seconds) and it is lazy loading friendly if we want to enable it globally. To be able to lazy load it, we encapsulate the binded calls in spacemacs/xxx functions which take care of firing the mode globally if required.
instead of at loading time which is useless :-)
Sometimes the spacemacs key bindings for ido-vertical did not work.
I had to explicitly enable projectile-enable-caching and explicitly start projectile-mode to initialize projectile-projects-cache
Also add ?r projectile-replace to commander
It is now part of the package
This is the command for helm-projectile
Diminish it to Ⓖ
guide-key does not play well with golden-ratio. also set delay to 1 seconds, can be configured in .spacemacs for shorter delay
(add-hook 'inferior-python-mode-hook '(lambda () | ||
(smartparens-mode) | ||
(evil-search-highlight-persist -1) | ||
(global-evil-search-highlight-persist -1))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why globally disable it ?
I submitted a PR to remove the :global t
in evil-search-highlight-persist. I hope it will be merged soon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I disabled globally just because otherwise evil-search-highlight-persist
still takes in effect.
If evil-search-highlight-persist
is not enabled globally by default, then there's no need to disable it =), I took it out.
Check with the new version which should be available soon. It fixes some activation issues and remove the global property. |
Great ! 👍 Cherry-picked into |
I have made two improvement for python inferior buffer:
comint-clear-buffer
buffer from http://emacsredux.com/blog/2015/01/18/clear-comint-buffers/, and bind it toC-l
in python inferior buffer.C-r
to search inferior buffer commands backward, and disableevil-search-highlight-persist
in python inferior buffer. Highlighting matched history in a repl buffer is quite annoying.Both
C-l
andC-r
are compliant with the keys in a shell terminal.Since both updates are quite small, I kept them in one PR, although they're separate commits in git history. Let me know if I need to make separate PR. Thanks.