[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windo
From: |
Eli Zaretskii |
Subject: |
bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows |
Date: |
Sun, 26 Mar 2023 07:19:12 +0300 |
> Cc: 62427@debbugs.gnu.org
> From: Juri Linkov <juri@linkov.net>
> Date: Sat, 25 Mar 2023 21:14:24 +0200
>
> > I've noticed that when I call #'tab-bar-new-tab while I'm in a
> > side-window that has siblings, I get an error from
> > #'window--sides-check, which happens when #'tab-bar-new-tab calls
> > #'delete-other-windows. Here's an example of my problem:
> > ...
> > The attached patch fixes this issue. Could it be applied to the emacs-29
> > branch?
>
> Thanks. Unless Eli has objections, I'd like to push the patch
> to the emacs-29 branch since it correctly fixes the problem.
I'd like first to understand better the code involved and the
suggested change in it. Just reading the code of tab-bar-new-tab-to,
it looks strange. Why does tab-bar.el care about "window parameters
that can cause problems with 'delete-other-windows' and
'split-window'" to begin with? And why removing these parameters from
windows is TRT for tab-bar.el to resolve such problems, when it
doesn't really "own" those windows?
(Also, the patch's commit log message is not according to our
conventions, but that's a minor issue.)
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Benson Chu, 2023/03/24
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Juri Linkov, 2023/03/25
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows,
Eli Zaretskii <=
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Juri Linkov, 2023/03/27
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Eli Zaretskii, 2023/03/27
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Juri Linkov, 2023/03/27
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Eli Zaretskii, 2023/03/27
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Benson Chu, 2023/03/27
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Eli Zaretskii, 2023/03/28
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Benson Chu, 2023/03/28
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Eli Zaretskii, 2023/03/28
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Benson Chu, 2023/03/28
- bug#62427: tab-bar-new-tab-to now handles cases with multiple side-windows, Juri Linkov, 2023/03/30