Closed bmag closed 8 years ago
Sounds like an Emacs bug to me. If special care is taken to ensure that the clone-of
parameter is restored, why then isn't the same done for other parameters that should be preserved as well?
Please report this upstream and submit the debbugs ticket here. After finding out what the official stance of the Emacs developers is on this, I'll add a point on it to the README.
Opened bug report 23858: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=23858, lets see how it goes.
Thank you!
I've checked again and it appears as if they aren't thrilled of the idea of changing defaults, but Martin Rudalics isn't opposed to enhancing window-state-get
/window-state-put
to support all window parameters.
I see a few options for proceeding:
window-persistent-parameters
in the README (clean solution for the few that care)window-persistent-parameters
to an alist containing all the interesting parameters around everything using window-state-get
/window-state-put
(would magically work, but ignore a customized variable which sucks)window-persistent-parameters
for the users (plzno)As you could probably guess, option 2 sounds best to me.
As you could probably guess, option 2 sounds best to me.
Agreed, that's probably the best.
Great, I've added a notes section. For now it only contains this caveat, your provided customization sample and a link to this issue.
Reproduction guide:
M-:
)buff1 is displayed on the left in a side window.
(window-parameter mywin 'window-side)
returned value:
left
(window-parameter mywin 'window-side)
returned value:
nil
, meaning buff1's window is a normal window now, not a side window. My expectation is that buff1's window should remain a side window.To be fair, the same result happens if I replace step 4 with this step:
M-: (window-state-put (window-state-get mywin t) mywin)
Changing the value of
window-persistent-parameters
before step 4 solves my issue:I'm not sure why
window-side
andwindow-slot
are not persistent by default, there could be a reason I'm not aware of.I'm mentioning all this because I thought it could be helpful if eyebrowse modified
window-persistent-parameters
accordingly or suggested it as user configuration in the readme.