From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Dokos Subject: Re: [PATCH] Re: Latex Export: Place Caption Below Table Date: Mon, 03 Oct 2011 10:43:14 -0400 Message-ID: <701.1317652994@alphaville.dokosmarshall.org> References: <87zkhor9hb.fsf@ucl.ac.uk> <17486.1317238846@alphaville.dokosmarshall.org> <878vp2fqch.fsf@ucl.ac.uk> Reply-To: nicholas.dokos@hp.com Return-path: Received: from eggs.gnu.org ([140.186.70.92]:43145) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RAjjn-0001QQ-Li for emacs-orgmode@gnu.org; Mon, 03 Oct 2011 10:43:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RAjjj-0007bD-GL for emacs-orgmode@gnu.org; Mon, 03 Oct 2011 10:43:23 -0400 Received: from g6t0185.atlanta.hp.com ([15.193.32.62]:19018) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RAjjj-0007ay-Cr for emacs-orgmode@gnu.org; Mon, 03 Oct 2011 10:43:19 -0400 In-Reply-To: Message from Eric S Fraga of "Mon, 03 Oct 2011 09:10:06 BST." <878vp2fqch.fsf@ucl.ac.uk> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: "Thomas S. Dye" , emacs-orgmode@gnu.org, Jakob Lombacher Cc: nicholas.dokos@hp.com Eric S Fraga wrote: > tsd@tsdye.com (Thomas S. Dye) writes: > > > Nick Dokos writes: > > >> The following patch (deliberately hidden as a binary octet-stream to keep it > >> out of patchwork) will do that - but IMO, it would be better to have yet another > >> user-settable option to control the placement. > >> > >> Nick > >> > >> > > > > Aloha all, > > > > The inlined patch introduces a variable > > org-export-latex-table-caption-above to control the placement of table > > captions. Thanks to Nick Dokos for leading the way on this. > > > > Tom > > > thanks for doing this -- I was swamped last week so had no chance of > following the thread until today! > > One comment on the last version of the patch: would it not be better for > the variable to be a defcustom variable instead of defvar? I agree - being able to customize variables without resorting to lisp is generally preferable: it is friendlier to newcomers and it allows even experienced users to find the various knobs. If the patch is deemed acceptable, it might also be nice to be able to set the variable per-file through the OPTIONS line, but that can certainly wait. > Otherwise, > the patch looks fine to me although I'm unlikely to use the new > functionality ;-) Unless you try to publish an article in a journal with a perverse layout policy :-) I applied the patch and took it for a short spin. git complained about trailing whitespace: ,---- | /home/nick/Mail/inbox/1021:197: trailing whitespace. | (if (and floatp org-export-latex-table-caption-above) | /home/nick/Mail/inbox/1021:212: trailing whitespace. | (if (and floatp (not org-export-latex-table-caption-above)) | /home/nick/Mail/inbox/1021:225: trailing whitespace. | (if (not org-export-latex-table-caption-above) tbl) | /home/nick/Mail/inbox/1021:231: trailing whitespace. | (if org-export-latex-table-caption-above tbl) | warning: 4 lines add whitespace errors. `---- but other than that the patch worked fine. Thanks, Tom! Nick