--- Begin Message ---
Subject: |
30.0.50; image-auto-resize :type has choices in wrong order |
Date: |
Sun, 10 Sep 2023 10:40:45 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 |
The :type for image-auto-resize looks like this:
:type '(choice (const :tag "No resizing" nil)
(const :tag "Fit to window" fit-window)
(other :tag "Scale down to fit window" t)
(number :tag "Scale factor" 1))
The `other' choice should come last. Otherwise Custom thinks that
that's the selected option, even when the value of image-auto-resize is
a number.
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#65852: 30.0.50; image-auto-resize :type has choices in wrong order |
Date: |
Sun, 10 Sep 2023 08:38:03 -0700 |
Mauro Aranda <maurooaranda@gmail.com> writes:
> On 10/9/23 11:24, Stefan Kangas wrote:
> >> I worked on something like that for elint.el, based on the wishlist on
> >> Bug#2591: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=2591
> >>
> >> But I know it might be better to make it so that the byte-compiler
> >> warns, so I never posted it.
> >
> > I'm not sure if elint.el is used all that much, but I might be wrong.
>
> A shame, if you ask me. It helped me find two more options with this
> problem: font-lock-verbose and message-openpgp-header. I attach an
> updated patch, but let me know if you prefer a separate bug report.
Thanks, pushed to master.
--- End Message ---