Subject: bug#65468: 29.1; describe-theme fails to describe some themes not loaded

From: Eli Zaretskii <[email-masked]>
Date: Wed, 23 Aug 2023 14:57:10 +0300

> Cc: [email-masked]
> Date: Wed, 23 Aug 2023 06:53:03 -0300
> From: Mauro Aranda <[email-masked]>
> 
> Thierry Volpiatto <[email-masked]> writes:
> 
>  > This because `describe-theme-1` is not looping in buffer to find
>  > `deftheme` definition.
> 
> It is a convention that the first form should be a call to deftheme.
> But it seems built-in themes have deviated from that convention.  And
> other themes might as well, I don't know.
> 
>  >
>  > Try (describe-theme 'leuven) to reproduce (if not already loaded of 
> course).
>  >
> 
> At least for the leuven themes, it should be easy to make them follow
> the convention.

I think we should indeed fix the themes that come with Emacs.