Re: [PATCH] LaTeX export: arbitrary float environments
Hi Thomas, "Thomas S. Dye" writes: > LaTeX users are able to define arbitrary float types, e.g. with the > float package. The attached patch makes them accessible from Org > mode. Applied in master with minor enhancements in the commit message and the code (removing a useless check against the value of float.) Thanks, -- Bastien
Re: [PATCH] LaTeX export: arbitrary float environments
Thomas S. Dye writes: > Aloha Timothy, :waves: > Sorry for the clumsy patch, which I guess would also benefit from an addition > to > the manual, as well? No problem, we all start somewhere :) (and I know I'm still making mistakes) An update to the manual to describe the changed behaviour would be good. If you'd like you could wait till a core maintainer says this looks good before going to that effort, or you could add it in now so it's all in a single patch --- as long as something happens. > Larger question: do we really want to tinker with ob-latex in this way? Or, > should changes like this patch follow a path indicated by Tim Cross and into > their own package, say ob-latex-ex, which might someday replace ob-latex if it > proved useful and stable? I don't think LaTeX support is likely to leave Org any time soon, so at least until a core maintainer tells you otherwise I'd be inclined to proceed with your current patch. If this was a huge/transformational change I may be more uncertain, but this looks fairly simple to me at least. -- Timothy > All the best, > Tom > > Timothy writes: > >> Hi Thomas, >> >> On the surface, this looks reasonable to me :) >> >> Just commenting on some technicalities with the patch itself: >> - In ORG-NEWS it would be good to wrap the content over multiple lines >> instead of having a single 270 char line :) >> - You seem to have an anomalous change to the ob-python :return entry >> - I don't think your patch subject follows the convention for Org, it >> should be: >> "main file/feature: overall change summary" >> so, something like >> "ox-latex: allow for arbitrary float environments" >> rather than >> "LaTeX export: arbitrary float environments" >> >> Thanks for the patch :) >> >> Timothy
Re: [PATCH] LaTeX export: arbitrary float environments
Aloha Timothy, Thanks for your kind response. Sorry for the clumsy patch, which I guess would also benefit from an addition to the manual, as well? Larger question: do we really want to tinker with ob-latex in this way? Or, should changes like this patch follow a path indicated by Tim Cross and into their own package, say ob-latex-ex, which might someday replace ob-latex if it proved useful and stable? All the best, Tom Timothy writes: Hi Thomas, On the surface, this looks reasonable to me :) Just commenting on some technicalities with the patch itself: - In ORG-NEWS it would be good to wrap the content over multiple lines instead of having a single 270 char line :) - You seem to have an anomalous change to the ob-python :return entry - I don't think your patch subject follows the convention for Org, it should be: "main file/feature: overall change summary" so, something like "ox-latex: allow for arbitrary float environments" rather than "LaTeX export: arbitrary float environments" Thanks for the patch :) Timothy -- Thomas S. Dye https://tsdye.online/tsdye
Re: [PATCH] LaTeX export: arbitrary float environments
Hi Thomas, On the surface, this looks reasonable to me :) Just commenting on some technicalities with the patch itself: - In ORG-NEWS it would be good to wrap the content over multiple lines instead of having a single 270 char line :) - You seem to have an anomalous change to the ob-python :return entry - I don't think your patch subject follows the convention for Org, it should be: "main file/feature: overall change summary" so, something like "ox-latex: allow for arbitrary float environments" rather than "LaTeX export: arbitrary float environments" Thanks for the patch :) Timothy
[PATCH] LaTeX export: arbitrary float environments
Aloha all, LaTeX users are able to define arbitrary float types, e.g. with the float package. The attached patch makes them accessible from Org mode. This is a follow on to my efforts several years ago to support the Tufte-LaTeX package in Org mode, and a suggestion at the time (by Rasmus iirc) to implement an :environment attribute for LaTeX export. This patch achieves a similar goal, but is a bit lighter imo. Let me know if you have questions. All the best, Tom >From 5154901b781f93d08851f96431c976f010fc420c Mon Sep 17 00:00:00 2001 From: "Thomas S. Dye" Date: Sun, 4 Apr 2021 08:11:40 -1000 Subject: [PATCH] LaTeX export: arbitrary float environments * lisp/ox-latex.el (`org-latex--inline-image', `org-latex--decorate table'): recognize arbitrary :float value. LaTeX users are able to define arbitrary float types. This patch makes them accessible from Org mode. * etc/ORG-NEWS: Announce new :float capability. --- etc/ORG-NEWS | 6 +- lisp/ox-latex.el | 16 +++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS index 9fc126b2f..cdfb1c727 100644 --- a/etc/ORG-NEWS +++ b/etc/ORG-NEWS @@ -12,6 +12,10 @@ Please send Org bug reports to mailto:emacs-orgmode@gnu.org. * Version 9.5 (not yet released) ** New options and settings +*** LaTeX attribute ~:float~ now passes through arbitrary values + +LaTeX users are able to define arbitrary float types, e.g. with the float package. The Org mode LaTeX exporter is now able to process and export arbitrary float types. The user is responsible for ensuring that Org mode configures LaTeX to process any new float type. + *** Option ~org-hidden-keywords~ now also applies to #+SUBTITLE: The option ~org-hidden-keywords~ previously applied @@ -106,7 +110,7 @@ behavior. By default ox-html now inlines webp images. ** New features -*** =ob-python= improvements to =:return= header argument +*** =ob-python= improvements to =:return= header argument The =:return= header argument in =ob-python= now works for session blocks as well as non-session blocks. Also, it now works with the diff --git a/lisp/ox-latex.el b/lisp/ox-latex.el index 376d27a07..514801d7c 100644 --- a/lisp/ox-latex.el +++ b/lisp/ox-latex.el @@ -2377,6 +2377,7 @@ used as a communication channel." ((string= float "sideways") 'sideways) ((string= float "multicolumn") 'multicolumn) ((and (plist-member attr :float) (not float)) 'nonfloat) +(float float) ((or float (org-element-property :caption parent) (org-string-nw-p (plist-get attr :caption))) @@ -2470,6 +2471,18 @@ used as a communication channel." nil t ;; Return proper string, depending on FLOAT. (pcase float + ((and (pred stringp) env-string) + (format "\\begin{%s}%s +%s%s +%s%s +%s\\end{%s}" + env-string + placement + (if caption-above-p caption "") + (if center "\\centering" "") + comment-include image-code + (if caption-above-p "" caption) + env-string)) (`wrap (format "\\begin{wrapfigure}%s %s%s %s%s @@ -3200,7 +3213,7 @@ centered." (defun org-latex--decorate-table (table attributes caption above? info) "Decorate TABLE string with caption and float environment. -ATTRIBUTES is the plist containing is LaTeX attributes. CAPTION +ATTRIBUTES is the plist containing LaTeX attributes. CAPTION is its caption, as a string or nil. It is located above the table if ABOVE? is non-nil. INFO is the plist containing current export parameters. @@ -3211,6 +3224,7 @@ Return new environment, as a string." (cond ((and (not float) (plist-member attributes :float)) nil) ((member float '("sidewaystable" "sideways")) "sidewaystable") ((equal float "multicolumn") "table*") + (float float) ((or float (org-string-nw-p caption)) "table") (t nil (placement -- 2.25.1 -- Thomas S. Dye https://tsdye.online/tsdye