[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#59862: quit-restore per window buffer
From: |
martin rudalics |
Subject: |
bug#59862: quit-restore per window buffer |
Date: |
Sat, 15 Jun 2024 10:41:54 +0200 |
User-agent: |
Mozilla Thunderbird |
+ (< (seq-count (lambda (w) (window-parameter w 'quit-restore))
The entire clause in 'quit-restore-window' governed by
(eq (nth 1 quit-restore) 'tab)
needs a comment on what it is supposed to restore. Basically, you check
here whether WINDOW is the only window with a 'quit-restore' parameter.
Is it that what you really want? If a window has a 'quit-restore'
parameter whose second element is 'tab' does that mean that all other
windows on the same frame must also have such a parameter? What happens
if there are other windows without a 'quit-restore' parameter?
As designed, the 'quit-restore' parameter can cause the deletion of its
window only if there are other windows on the same frame. It can cause
the deletion of its frame only if there is no other window on its frame.
Are these principles preserved by your patch?
> + (window-list-1 nil 'nomini))
I suppose that nil is not correct here when WINDOW is not on the
selected frame. I'd rather use that window instead here.
> + 2))
> (tab-bar-close-tab)
> ;; If the previously selected window is still alive, select it.
> (window--quit-restore-select-window quit-restore-2))
martin
- bug#59862: quit-restore per window buffer, (continued)
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/03
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/03
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/03
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/04
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/05
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/11
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/12
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/13
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/13
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/14
- bug#59862: quit-restore per window buffer,
martin rudalics <=
- bug#59862: quit-restore per window buffer, Juri Linkov, 2024/06/16
- bug#59862: quit-restore per window buffer, martin rudalics, 2024/06/17