Subject: bug#57639: [PATCH] Add new command 'toggle-theme'

From: Protesilaos Stavrou <[email-masked]>
Date: Mon, 10 Oct 2022 04:17:38 +0300

> From: Stefan Kangas <[email-masked]>
> Date: Sun,  9 Oct 2022 22:32:49 +0200
>
> Philip Kaludercic <[email-masked]> writes:
>
>> --8<---------------cut here---------------start------------->8---
>> ;;;###autoload
>> (when (and (boundp 'custom-theme-load-path)
>>            load-file-name)
>>   ;; Add theme folder to `custom-theme-load-path' when installing over MELPA.
>>   (add-to-list 'custom-theme-load-path
>>                (file-name-as-directory (file-name-directory load-file-name))))
>> --8<---------------cut here---------------end--------------->8---
>>
>> from leuven-theme.el to be removed.
>
> Why would the above be necessary for leuven-theme.el in particular?
>
> I've never had a problem loading themes installed from MELPA, so I don't
> understand why it should need any special treatment.

Note that the modus-themes.el have this:

    ;;;###autoload
    (when load-file-name
      (let ((dir (file-name-directory load-file-name)))
        (unless (equal dir (expand-file-name "themes/" data-directory))
          (add-to-list 'custom-theme-load-path dir))))

It is (was?) necessary to make the theme available when installing with
package.el (GNU ELPA).

-- 
Protesilaos Stavrou
https://protesilaos.com