Re: [PATCH] Address issue with tab-bar

[James N . V . Cash]

Juri Linkov <juri@linkov.net> writes:

> But according to the documentation of 'tab-bar-show':
>
>   "If the value is =E2=80=981=E2=80=99, then hide the tab bar when it has=
 only one tab,
>   and show it again once more tabs are created."
>
> tab-bar-close-tab should hide the tab-bar on the selected frame
> with only one tab.
>
> So the problem is that tab-bar-new-tab doesn't show the tab-bar again
> when there are more than one tab.

Ah, in that case, I think we can just remove the first case in the cond
in tab-bar-new-tab-to; simple patch attached

> It seems this is a consequence of bug#42052 that tried to implement
> frame-local tab-bar where tab-bar-mode is always true, but it shows/hides
> frame-local tab-bars depending on the frame-parameter 'tab-bar-lines'.

(I think that was me, whoops)

diff --git a/lisp/tab-bar.el b/lisp/tab-bar.el
index d8f932e7a4..e4b3c8cf19 100644
--- a/lisp/tab-bar.el
+++ b/lisp/tab-bar.el
@@ -801,7 +801,6 @@ tab-bar-new-tab-to
                           (nth to-index tabs)))
 
     (cond
-     (tab-bar-mode)
      ((eq tab-bar-show t)
       (tab-bar-mode 1))
      ((and (natnump tab-bar-show)

P.S. Juri, sorry for the double-send, I forgot to CC the list the first time