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

Fix behavior of cider-switch-to-current-repl-buffer #1153

Conversation

whysoserious
Copy link
Contributor

This setting in my config file

(setq cider-repl-display-in-current-window t)

breaks behavior of cider-repl-display-in-current-window command. See it in action. Each change of window layout or buffer in the movie below is an effect of hitting C-c C-z:
lol

This PR hopefully fixes that (watch out - this is my first elisp PR ever).

Software versions:

; CIDER 0.9.1snapshot (package: 20150618.2308) (Java 1.8.0_25, Clojure 1.6.0, nREPL 0.2.8)

GNU Emacs 24.4.1 (x86_64-apple-darwin13.4.0, NS apple-appkit-1265.21) of 2014-10-21 on builder10-9.porkrind.org

@bbatsov
Copy link
Member

bbatsov commented Jun 20, 2015

Hmm, another epic oversight on my part. Seems that I originally implemented this to affect only the REPL buffer creation... The fix looks good. Just add a changelog entry and we're good to go.

whysoserious pushed a commit to whysoserious/cider that referenced this pull request Jun 20, 2015
whysoserious pushed a commit to whysoserious/cider that referenced this pull request Jun 20, 2015
@whysoserious
Copy link
Contributor Author

Done.

@bbatsov
Copy link
Member

bbatsov commented Jun 20, 2015

@whysoserious Please, squash the two commits together. No need to have them separately.

@whysoserious whysoserious force-pushed the fix-cider-switch-to-current-repl-buffer branch from efc2299 to fe6c136 Compare June 20, 2015 20:51
@whysoserious
Copy link
Contributor Author

Done.

@bbatsov
Copy link
Member

bbatsov commented Jun 20, 2015

👍 Thanks!

bbatsov added a commit that referenced this pull request Jun 20, 2015
…t-repl-buffer

Fix behavior of `cider-switch-to-current-repl-buffer`
@bbatsov bbatsov merged commit 8bb3890 into clojure-emacs:master Jun 20, 2015
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.

None yet

2 participants