GNU bug report logs - #76394
mouse-face property not working in tab-bar

Previous Next

Full log

View this message in rfc822 format

From: Ship Mints <shipmints <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 76394 <at> debbugs.gnu.org
Subject: bug#76394: mouse-face property not working in tab-bar
Date: Sun, 2 Mar 2025 14:37:02 -0500

[Message part 1 (text/plain, inline)]

On Sun, Mar 2, 2025 at 12:52 PM Juri Linkov <juri <at> linkov.net> wrote:

> > It seemed natural to add tab-bar-tab-name-format-mouse-face function to
> the
> > tab-bar-tab-name-format-functions list.  See attached.
>
> Thanks, I've tested your patch, and it works great!
>
> > +(defface tab-bar-tab-highlight
> > +  '((t :inherit highlight))
> > +  "Tab bar face for highlighting."
> > +  :version "31.1"
> > +  :group 'tab-bar-faces)
>
> Please use exactly the same style as in 'tab-line-highlight'.
>

Done.

> +(defun tab-bar-tab-name-format-mouse-face (name tab _i)
>
> The arg 'tab' needs an underscore since compilation gives the
> warning: Unused lexical argument `tab'.
>

Done.  I almost always run the byte compiler to check my changes but
neglected this time.

> +      /* display_tab_bar does not yet support R2L.  */
> > +      eassert (row->reversed_p);
>
> Is this still needed?
>

Eli asked for that.

While we're at this, should we implement the equivalent of
tab-line-close-highlight in tab-bar?  I can do that before I share the next
patch.

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.