Re: Tabs

[Eli Zaretskii]

> Date: Mon, 14 Oct 2019 09:37:07 +0300
> From: Eli Zaretskii <address@hidden>
> Cc: address@hidden, address@hidden
> 
> > The problem is that tab-bar.el is pre-loaded by loadup.el and
> > dumped with the value of data-directory from the source location.
> > So running from the dump file even in the install directory
> > still uses source data-directory.
> 
> Why can't you compute the value at run time, rather than at dump time?
> Like the first time the tab bar is activated?

I see that you have done so, thanks.

However, there should be no need to explicitly expand-file-name for
the image files, as the documentation of the :file attribute says:

  ‘:file FILE’
       This says to load the image from file FILE.  If FILE is not an
       absolute file name, it is expanded relative to the ‘images’
       subdirectory of ‘data-directory’, and failing that, relative to the
       directories listed by ‘x-bitmap-file-path’ (*note Face
       Attributes::).

So by explicitly calling expand-file-name, you introduce a subtle
misfeature, whereby x-bitmap-file-path will not be consulted.