Re: [BUG] Image width specifications like :width 300px not working

[Ihor Radchenko <yantar92@posteo.net>]

Karthik Chikmagalur <karthikchikmagalur@gmail.com> writes:

> Due to the change in 50be048ad0cbe0e9150ca2351b894b041ae7cf49, image
> display specifications like 
>
> #+attr_org: 300px
>
> don't work.  From the commit message, it looks like the intent was to
> ignore specifications like :width 4in.  I'm not sure if ignoring "px"
> specifications was intentional.
>
> If this is not a bug and width should be specified as an integer
> (i.e. 300 instead of 300px), then the manual needs to be updated.
> In (info "(org) Images") I still see examples like
>
>           #+ATTR_HTML: :width 300px

I agree that it is a regression from the existing behavior.
(just as trating 4in and pixels was a regression from earlier behavior)

Probably, we can extend the regexp to treat things like px/pt as pixel
width as well.

I suggest doing it in a separate patch rather than making many changes
together with introducing a new feature as you suggested. Having too
many things in a single commit can lead to confusion. 

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>