Re: [PATCH] Fix regex for determining image width from attribute

[Max Nikulin]

On 08/05/2024 17:54, Ihor Radchenko wrote:
> Max Nikulin writes:
>
> > #+attr_html: :alt Image width test
> > #+attr_beamer: :width \linewidth
> > #+attr_latex: :width +.5\textwidth
> > #+attr_md: :width 75%
> > [[file:babelfish.png]]
> >
> > - I do not mind that just "\linewidth" is ignored.
> > - The case of "+.5" should either be supported or at least documented
> > since it is a valid floating number.
>
> Was it supported before?

No. It is my expectation based on earlier discussions that fractional 
numbers were added namely for LaTeX. I used leading dot in 
\includegraphics[width=...]{}, so I was surprised that Org ignores it. 
Explicit "+" is just generalization, this form is supported by LaTeX and 
elisp.

> > - It is really confusing that #+attr_html casts shadow on #+attr_md.
>
> No, it is not. I see no reason to prioritize markdown attributes.

Above #+attr_html does not specify :width while #+attr_md has a valid 
value. It is the reason why I do like current behavior.

> > > +++ b/etc/ORG-NEWS
> >
> > I think, it is better to avoid "is ignored" here.
>
> May you convert your suggestion into a patch?

See the attachment.

0001-ORG-NEWS-Reword-inline-image-width-note.patch
Description: Text Data