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

Moving to a complex workspace using eyebrowse while using golden-ratio-mode results in error #72

Open
topher200 opened this issue Nov 19, 2016 · 4 comments

Comments

@topher200
Copy link

Description :octocat:

Creating or moving to a eyebrowse workspace with a complex window arrangement while in golden-ratio-mode results in error.

Turning off golden-ratio-mode removes the error.

Observed on Spacemacs' master and develop. Observed on base Emacs install with no Spacemacs or other libraries pre-loaded.

Reproduction guide 🪲

  1. Clear out your emacs configuration by deleting or moving your .emacs.d/
  2. Start Emacs
  3. Enable Marmalade repo (https://marmalade-repo.org/#download)
  4. Install eyebrowse using install-package
  5. Install el-get (https://github.com/dimitri/el-get)
  6. Install golden-ratio-mode using el-get-install golden-ratio-mode
  7. M-x eyebrowse-mode Turn on eyebrowse mode
  8. M-x golden-ratio-mode Turn on golden-ratio mode
  • Step 8B: Close all windows but one, using C-x 0. Load your *scratch* buffer the one remaining window
  1. C-x 3 Split window on the right
  2. C-x 2 Split window on the bottom
  3. C-x 3 Split window on the right
  4. ✅ There should be four windows
  5. M-x eyebrowse-create-window-config Create a new workspace using eyebrowse

Observed behaviour: 👀 💔
The new workspace is created!

However, we stay in our original workspace. One of the windows has disappeared from our original workspace; we now have three windows.

This error appears in the status bar:

window--state-put-1: Cannot resize window #<window 56 on  *temp*>

Expected behaviour: ❤️ 😄
The new workspace is created. We move to the new workspace. Our original workspace is unmodified.


I originally reported this to the Spacemacs repo here: syl20bnr/spacemacs#7779

They said to file a bug upstream, so I filed one with eyebrowse-mode here: wasamasa/eyebrowse#56

The eyebrowse-mode developer took a look at the issue. He reproduced the issue and made some fixes to eyebrowse-mode (releasing version 0.7.2). However, he and I both agree that his fixes do not solve the issue - it's still able to be reproduced. He thinks underlying issue should be solved inside of golden-ratio-mode and has closed the issue in eyebrowse; see his comment here: wasamasa/eyebrowse#56 (comment)

@chu-
Copy link

chu- commented Feb 24, 2017

Same problem.

@ChrisSalisbury
Copy link

I haven't thoroughly tested it, but PR #66 seems to solve this problem for me in case anyone else is looking for an easy solution. If that PR could be properly tested and merged it could very well solve this issue.

@azzamsa
Copy link

azzamsa commented Oct 8, 2018

Same problem here.

@ChrisSalisbury no. if you open more than 2 windows the problem still occurred.

Steps:

  1. open eyebrowse workspace 1, 2 and 3
  2. open 1 window in w1, 3 windows in w2 and 3 windows in w3.
  3. try to switch workspace. The order of buffer will messed up and the error occurred again. (the error doesn't happen in w1 because it's only contain 1 window)
window--state-put-1: Cannot resize window #<window 69 on  *temp*>
window--state-put-1: Cannot resize window #<window 78 on  *temp*>

@llcc
Copy link

llcc commented Oct 30, 2018

I also tested PR #66, it doesn't work well. @ChrisSalisbury

Hope this issue can be solved.

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

5 participants