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

Layout breaks in Emacs 25.1 #51

Open
kini opened this issue Nov 8, 2016 · 3 comments
Open

Layout breaks in Emacs 25.1 #51

kini opened this issue Nov 8, 2016 · 3 comments

Comments

@kini
Copy link

kini commented Nov 8, 2016

Steps to reproduce:

  • emacs -Q
  • require shell-pop
  • C-x 3 to split into left and right windows
  • M-x shell-pop twice

Expected behavior:

After the first M-x shell-pop, a window appears at the bottom of the frame containing a shell. After the second M-x shell-pop, that window goes away and the layout of the frame returns to what it was previously (two windows side by side).

Actual behavior:

After the first M-x shell-pop, a window appears at the bottom of the frame, but the shell is loaded into the window on the right. After the second M-x shell-pop, the right side window is removed and what remains is a large top window and a small bottom window.

After the first M-x shell-pop:

screenshot from 2016-11-08 10-33-25

After the second M-x shell-pop:

screenshot from 2016-11-08 10-33-30

@kini kini changed the title Layout breaks in Emacs 25.1.1 Layout breaks in Emacs 25.1 Dec 2, 2016
@MRFaria
Copy link

MRFaria commented Apr 26, 2017

I think this happens because by default emacs25 changed the shell buffers to open in a new window instead of the same one.

Not sure how to fix shell-pop, but you can get the correct behavior with:
(push (cons "\\*shell\\*" display-buffer--same-window-action) display-buffer-alist)

Which makes the shell buffer open in the same window, which is what shell-pop expects

@rafipanoyan
Copy link

I have the same issue with Emacs 25.1.1

@articuluxe
Copy link

If anyone happens to use shackle, the following is an equivalent rule:
(add-to-list 'shackle-rules '("\\*shell\\*" :regexp t :same t))

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

4 participants