emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: "Thomas S. Dye" <tsd@tsdye.online>
To: Kyle Meyer <kyle@kyleam.com>
Cc: emacs-orgmode@gnu.org, Pablo Palazon <pablo.palazon@gmail.com>,
	Louis Gostiaux <louis.gostiaux@free.fr>,
	"Thomas S. Dye" <tsd@tsdye.online>
Subject: Re: [PATCH] Add margin option to float for figure in ox-latex.el
Date: Sat, 16 May 2020 08:00:13 -1000	[thread overview]
Message-ID: <87wo5b7ob6.fsf@tsdye.online> (raw)
In-Reply-To: <87o8qoqtau.fsf@kyleam.com>

Aloha Kyle,

This patch produces a LaTeX environment, marginfigure, that isn't 
part of the standard.  AFAIK, marginfigure is defined in the 
sidenotes package and separately in the tufte-latex class, neither 
of which Org mode loads by default.  If the patch is applied, then 
one of these packages should be added to the list of default LaTeX 
packages so ox-latex doesn't export code it is unable to compile.

My takeaway from the discussion surrounding ox-tufte-latex several 
years ago is that support for non-standard LaTeX constructs should 
not be part of Org mode core because they complicate maintenance 
unduly.

I haven't used ox-tufte-latex since I last used paper handouts at 
a presentation several years ago, so I don't know if it still 
works.  However, it really was cobbled together, a process that 
involved copying big chunks of Org mode code.  I knew then this 
approach is sub-optimal, but never found the time to puzzle out 
how to turn it into advice that could sit on top of a possibly 
changing code base.

Also, FYI, I found the tufte-latex class a bit difficult to use. 
Objects in the margin can easily overwrite one another and the 
author has to fix these manually, which can be tricky.  The upside 
is that when everything is right, the output can be terrific to 
look at.

All the best,
Tom

Kyle Meyer writes:

> Pablo Palazon writes:
>
>> I've created a path to add a new option to float properties for 
>> figures on
>> latex. This is my first change for org-mode, and I don't really 
>> sure if
>> this is the correct way to do it.
>
> Thank you!
>
>> * lisp/ox-latex.el (org-latex--inline-image): Include margin 
>> option
>> to create marginfigure environment for figures. It's useful for 
>> tufte
>> latex class, where with this environment shows the figure in 
>> the margin.
>
> I wondered if something like this had been proposed or discussed 
> before.
> Searching the list, I see a patch [0] very similar to yours from 
> a
> couple of months back that didn't get a response (author added 
> to cc).
>
> Further back, there is a thread about an exporter Thomas Dye 
> (+cc) wrote
> to handle marginfigure and some other Tufte-y things [1].  It 
> looks like
> the code is available at 
> <https://github.com/tsdye/tufte-org-mode>,
> though I'm not sure if it still works with the current Org.
>
> As for the proposed patch, while I think the specific code 
> change itself
> looks fine, skimming through the above thread makes me think 
> that adding
> marginfigure to ox-latex.el without considering similar cases 
> may not be
> the right approach.  What do others think?
>
>
> [0]: 
> https://yhetil.org/orgmode/35aac187-b751-5723-0f15-be6605fb8ea2@free.fr/
> [1]: https://yhetil.org/orgmode/m2h9hsgdo2.fsf@tsdye.com/


--
Thomas S. Dye
https://tsdye.online/tsdye


  parent reply	other threads:[~2020-05-16 18:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-14 16:06 [PATCH] Add margin option to float for figure in ox-latex.el Pablo Palazon
2020-05-15 12:21 ` Eric S Fraga
2020-05-15 17:02   ` Pablo Palazon
2020-05-16  6:37 ` Kyle Meyer
2020-05-16  7:17   ` Louis Gostiaux
2020-05-16 16:46     ` Kyle Meyer
2020-05-16 18:00   ` Thomas S. Dye [this message]
2020-05-16 20:35     ` Kyle Meyer
2020-05-17 16:48       ` Thomas S. Dye
2020-05-23  9:12         ` Bastien
     [not found] <mailman.39.1589558407.15748.emacs-orgmode@gnu.org>
2020-05-15 16:47 ` Pablo Palazon

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=87wo5b7ob6.fsf@tsdye.online \
    --to=tsd@tsdye.online \
    --cc=emacs-orgmode@gnu.org \
    --cc=kyle@kyleam.com \
    --cc=louis.gostiaux@free.fr \
    --cc=pablo.palazon@gmail.com \
    /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).