[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] ox-latex: provide width and height options for images
From: |
aaronecay |
Subject: |
Re: [O] [PATCH] ox-latex: provide width and height options for images |
Date: |
Tue, 05 Mar 2013 23:02:05 -0500 |
User-agent: |
Notmuch/0.15.2+33~g0c0a530 (http://notmuchmail.org) Emacs/24.3.50.1 (x86_64-unknown-linux-gnu) |
2013ko otsailak 27an, Nicolas Goaziou-ek idatzi zuen:
> Thank you for your patch. Here are a few comments.
Thank you for the comments! I think the patch is simpler and better
thanks to them.
>
>> These are implemented with \resizebox, and thus are uniform across
>> different types of image inclusion (\includegraphics, \input of tikz
>> images). This differs from the older way of using width and height
>> optional args to \includegraphics.
>
> I tend to agree with Rasmus. It would be better to keep height and
> width options in \includegraphics when possible.
Okay.
>
>> Thus, the default value for org-latex-image-default-options is left
>> untouched, to avoid breaking compatibility with older code. After a
>> transition period, the 0.9\linewidth value should be moved into
>> org-latex-image-default-width, and the -options variable set to the
>> empty string.
>
> We don't need this precaution. The exporter code for 8.0 introduced
> many incompatibilities already. Also, this one is easy to discover.
Okay.
>
> I think it's a good step forward. It will need to be documented in the
> comments at the beginning of ox-latex.el, where all attributes
> properties relative to different syntactical elements are explained.
Okay – see what you think of the documentation in the new patch.
>
> Here, you can obtain \resizebox{!}{!}{%s}, which is wrong, isn't it?
No...it is a no-op. That said, the code should no longer do this.
>
> This needs to be changed as these options would interfere with :width
> argument. For example, (eq float 'float) could set :width property if
> it is undefined. Obviously, this means the check has to be done before
> WIDTH and HEIGHT strings are built.
This should now be handled. The new patch will follow this email. In
related news, I have completed the FSF copyright assignment process, so
the patch can be merged whenever it is deemed ready.
Thanks again,
--
Aaron Ecay
- Re: [O] [PATCH] ox-latex: provide width and height options for images,
aaronecay <=