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

Behavior of the *cider-refresh-log* buffer is distracting #1254

Closed
luxbock opened this issue Aug 13, 2015 · 3 comments
Closed

Behavior of the *cider-refresh-log* buffer is distracting #1254

luxbock opened this issue Aug 13, 2015 · 3 comments

Comments

@luxbock
Copy link

luxbock commented Aug 13, 2015

I usually only care about the results of calling cider-refresh if there was a problem. If there were no problems I would much rather just get a message in the mini-buffer. Right now the behavior of the *cider-refresh-log* buffer is to pop-up in the other-window, which is annoying because I was already using that buffer for something else I care about. I would therefore propose the following changes to the behavior of running cider-refresh:

  • If there were no problems, log everything as usual but don't pop-up the log-buffer. Display a short message in the mini-buffer letting the user know that the reload was successful.
  • If the reload failed, display the log-buffer in the current-window. This way the user can see what failed, and get rid of the buffer by pressing q without having to change windows.
@bbatsov
Copy link
Member

bbatsov commented Aug 13, 2015

This is a duplicate of #1239 //cc @cichli

@bbatsov bbatsov closed this as completed Aug 13, 2015
@cichli
Copy link
Member

cichli commented Aug 17, 2015

@luxbock I've added cider-refresh-show-log-buffer now (defaults to nil) - let me know what you think next time you update?

@luxbock
Copy link
Author

luxbock commented Aug 20, 2015

Thanks! Works just as I had hoped.

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

No branches or pull requests

3 participants