emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Nicolas Goaziou <n.goaziou@gmail.com>
To: Rasmus <rasmus@gmx.us>
Cc: emacs-orgmode@gnu.org
Subject: Re: [ox-latex] Bad default value for image width?
Date: Wed, 10 Apr 2013 18:53:02 +0200	[thread overview]
Message-ID: <87k3oasic2.fsf@gmail.com> (raw)
In-Reply-To: <87y5cqiown.fsf@pank.iue.private> (rasmus@gmx.us's message of "Wed, 10 Apr 2013 13:52:08 +0200")

Hello,

Rasmus <rasmus@gmx.us> writes:

>> Set `org-latex-image-default-width' to "".
>
> But as my second email shows this still doesn't work with
> captions. . .
>
> This exmample:
>
> #+BEGIN_SRC org
> #+BEGIN_SRC emacs-lisp
> (make-local-variable 'org-latex-image-default-width)
> (setq org-latex-image-default-width "")
> #+END_SRC
>
> #+NAME:firm-ts-fig
> #+CAPTION: test
> [[file:figs/test.tikz]]
>
> no caption
> [[file:figs/test.tikz]]
> #+END_SRC

The problem is different here. More on this below.

>> This wouldn't help to solve the general problem. Though, you can change
>> it in your config to something else, as suggested above.
>
> I have.  But I don't think it's a sensible default.  It's against what
> your usual stand of Org not trying to be too clever.  You would also
> not put tables in resize boxes to make them "fit" on the page.

This is different. You always build tables from LaTeX so they are, in
a way, normalized. On the other hand, images can be imported from
various sources, with different sizes and resolutions.

> I think this is the solution to a general problem.  But I still find
> the default width to be wrong.  It only works as intended when I have
> a slightly too large picture.  And then, if anything, I'd want full
> page width.  
>
> It's a bad default for
>   - Picture smaller than textwidth
>   - Picture crafted to the document which shouldn't have a width.
>
> It's a decent solution for 
>   - pictures which are unintentionally larger than textwidth.  But IMO
>     it's my responsibility, and not Org's, to fix these
>     images/situations.

Exactly.

I'm not selling you the default value as the perfect solution: it isn't,
obviously. I even agree that in most situations, no default value is
better.

However, in my experience, the worst situation is the image (much)
larger than textwidth, which makes it difficult to even read the
produced document. It isn't as bad for enlarged small pictures.

So, the current default is good to control the worst case. And since
there is no decent solution in all situations, it makes sense here.
I think the same applies to default wrap width (.48\textwidth), which
ensures that the display will be coherent.

Now, the ".7\textwidth" for floats is harder to explain. I don't feel
very strong about it, and I don't mind removing it (meaning
`org-latex-image-default-width' would also apply
 to floats).

>> If "" is really read as "", """" will still be read as "\"\"\"\"", and
>> it will not be possible to obtain "\"\"".
>
> Fine with me. 
>
>> Another possibility is to read the empty string as the empty string
>> instead of nil, that is
>>
>>   #+attr_latex: :prop1 :prop2 1
>
> I like the "" better.  It works better with my "intuitive logic".

I have pushed a patch which should fix your initial problem about :width
"" not being taken into account.  Could you confirm the new behaviour
works as expected?


Regards,

-- 
Nicolas Goaziou

  reply	other threads:[~2013-04-10 16:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-07 20:17 [ox-latex] Bad default value for image width? Rasmus
2013-04-07 20:38 ` Rasmus Pank Roulund
2013-04-08 19:35 ` Bastien
2013-04-09 21:08   ` Nicolas Goaziou
2013-04-09 21:30     ` Bastien
2013-04-09 22:02 ` Nicolas Goaziou
2013-04-10 11:52   ` Rasmus
2013-04-10 16:53     ` Nicolas Goaziou [this message]
2013-04-10 17:23       ` Rasmus
2013-04-10 20:13         ` Nicolas Goaziou
2013-04-11 15:36           ` Bastien

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87k3oasic2.fsf@gmail.com \
    --to=n.goaziou@gmail.com \
    --cc=emacs-orgmode@gnu.org \
    --cc=rasmus@gmx.us \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).