Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hi Nick,
Nick Dokos wrote:
Nick Dokos nicholas.do...@hp.com wrote:
I think I read the whole thread but I'm still not clear on why
the change is needed. What goes awry with the environment instead
of the declaration?
Never mind - I didn't read the whole thread.
To summarize:
- \begin{center}...\end{center} adds (undesired) vertical space
- replacing by {\centering...\par} is the solution against that.
Best regards,
Seb
--
Sebastien Vauban
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Sebastien Vauban writes:
Hi Nick,
Nick Dokos wrote:
Nick Dokos nicholas.do...@hp.com wrote:
I think I read the whole thread but I'm still not clear on why
the change is needed. What goes awry with the environment instead
of the declaration?
Never mind - I didn't read the whole thread.
To summarize:
- \begin{center}...\end{center} adds (undesired) vertical space
- replacing by {\centering...\par} is the solution against that.
Or just \centering when contained in a figure or table environment
block so making the {...\par} unecessary.
Myles
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hello,
Myles English mylesengl...@gmail.com writes:
Nick, I think you are right about this, the modified patch at the end of
this email makes only the replacements that are necessary just to
replace the center environment within table blocks.
The other \begin{center}'s in the org-e-latex.el file may be changed
some other time, if needed.
Applied. Thank you.
Regards,
--
Nicolas Goaziou
[O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hi,
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el. I have tested
the export of a basic table and it works but not sure if all the
replacements are correct.
( BTW, this issue came to light using the nag package like this:
\RequirePackage[l2tabu, orthodox]{nag} )
Myles
From 54ae2245978fa755cd5daa483616ffdfa3e5ed0f Mon Sep 17 00:00:00 2001
From: Myles English mylesengl...@gmail.com
Date: Tue, 13 Nov 2012 14:14:56 +
Subject: [PATCH] org-e-latex: Tables get correct amount of vertical space
* contrib/lisp/org-e-latex.el: Replaced \begin{center} ..\end{center}
environments with \centering declarations.
Using both \begin{table} and \being{center} environments leads to double
the vertical space around the float, whereas \centering adds none.
TINYCHANGE
---
contrib/lisp/org-e-latex.el | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/contrib/lisp/org-e-latex.el b/contrib/lisp/org-e-latex.el
index 165beb3..c268c2c 100644
--- a/contrib/lisp/org-e-latex.el
+++ b/contrib/lisp/org-e-latex.el
@@ -550,14 +550,13 @@ in order to mimic default behaviour:
\(when tags
\(format \hfill{}textsc{:%s:}\
\(mapconcat 'identity tags \:\)
-\(format (concat \begin{center}\\n\
+\(format (concat \centering\\n\
\fbox{\\n\
\begin{minipage}[c]{.6textwidth}\\n\
\%s\\n\\n\
\rule[.8em]{textwidth}{2pt}\\n\\n\
\%s\
- \end{minipage}}\
- \end{center}\)
+ \end{minipage}}\)
full-title contents))
:group 'org-export-e-latex
:type 'function)
@@ -1086,7 +1085,7 @@ CONTENTS holds the contents of the center block. INFO is
a plist
holding contextual information.
(org-e-latex--wrap-label
center-block
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Clock
@@ -1508,15 +1507,14 @@ holding contextual information.
title
(when tags (format \\hfill{}\\textsc{:%s:}
(mapconcat 'identity tags :))
-(format (concat \\begin{center}\n
+(format (concat \\centering\n
\\fbox{\n
\\begin{minipage}[c]{.6\\textwidth}\n
%s\n\n
\\rule[.8em]{\\textwidth}{2pt}\n\n
%s
\\end{minipage}\n
-}\n
-\\end{center})
+})
full-title contents))
@@ -2302,14 +2300,13 @@ This function assumes TABLE has `org' as its `:type'
attribute.
(concat
(format \\begin{%s}%s\n float-env placement)
(if org-e-latex-table-caption-above caption )))
- (when org-e-latex-tables-centered \\begin{center}\n)
+ (when org-e-latex-tables-centered \\centering\n)
(format \\begin{%s}%s{%s}\n%s\\end{%s}
table-env
(if width (format {%s} width) )
alignment
contents
table-env)
- (when org-e-latex-tables-centered \n\\end{center})
(when float-env
(concat (if org-e-latex-table-caption-above caption)
(format \n\\end{%s} float-env
@@ -2351,7 +2348,7 @@ attribute.
(unless (= n 2)
(setq output (replace-match nil nil output)))
(if (not org-e-latex-tables-centered) output
- (format \\begin{center}\n%s\n\\end{center} output
+ (format \\centering\n%s output
Table Cell
--
1.8.0
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hi Myles,
Myles English wrote:
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el.
...
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Wouldn't you have to replace
\begin{center}
...
\end{center}
by
{\centering
...
}
? That is, add a group around?
Best regards,
Seb
--
Sebastien Vauban
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hi Seb,
Sebastien Vauban writes:
Hi Myles,
Myles English wrote:
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el.
...
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Wouldn't you have to replace
\begin{center}
...
\end{center}
by
{\centering
...
}
? That is, add a group around?
I don't think so, at least I have not come across that usage, and it
seems to work without. Do you know different?
Myles
Best regards,
Seb
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Myles English mylesengl...@gmail.com wrote:
Hi Seb,
Sebastien Vauban writes:
Hi Myles,
Myles English wrote:
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el.
...
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Wouldn't you have to replace
\begin{center}
...
\end{center}
by
{\centering
...
}
? That is, add a group around?
I don't think so, at least I have not come across that usage, and it
seems to work without. Do you know different?
If a \centering occurs at top-level in a latex document,
then *everything* after it will be centered: it's a declaration that
remains in force for the current group (which is the rest of the
document if it occurs at top-level).
\begin{center}...\end{center} is essentially {\centering ...} except
that it also starts a new paragraph.
It's not clear to me at least, that wholesale replacement is the correct
thing to do: it needs to be looked at on a case-by-case basis I think.
Nick
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Hi Myles and Nick,
Nick Dokos wrote:
Myles English mylesengl...@gmail.com wrote:
Sebastien Vauban writes:
Myles English wrote:
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el.
...
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Wouldn't you have to replace
\begin{center}
...
\end{center}
by
{\centering
...
}
? That is, add a group around?
I don't think so, at least I have not come across that usage, and it
seems to work without. Do you know different?
If a \centering occurs at top-level in a latex document,
then *everything* after it will be centered: it's a declaration that
remains in force for the current group (which is the rest of the
document if it occurs at top-level).
As confirmed by Nick, \centering just changes the layout for ever... up to the
end of the document.
Why does it work? I guess because there are grouping commands inserted by Org
at other places, that stop the propagation of the centering that far... But
it's dangerous to count on them, IMHO.
\begin{center}...\end{center} is essentially {\centering ...} except
that it also starts a new paragraph.
It's not clear to me at least, that wholesale replacement is the correct
thing to do: it needs to be looked at on a case-by-case basis I think.
I think that's a safe replacement (_with_ the grouping), and with something I
forgot about (well seen Nick!): the \par.
See, for example,
https://groups.google.com/forum/?fromgroups=#!topic/fr.comp.text.tex/0s4WfsIfmy8
for the answer of Manuel Pégourié-Gonnard, one reference wrt LaTeX questions:
replace the environment by {\centering ... \par}
Best regards,
Seb
--
Sebastien Vauban
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Sebastien Vauban wxhgmqzgw...@spammotel.com wrote:
I think that's a safe replacement (_with_ the grouping), and with something I
forgot about (well seen Nick!): the \par.
See, for example,
https://groups.google.com/forum/?fromgroups=#!topic/fr.comp.text.tex/0s4WfsIfmy8
for the answer of Manuel Pégourié-Gonnard, one reference wrt LaTeX questions:
replace the environment by {\centering ... \par}
I think I read the whole thread but I'm still not clear on why
the change is needed. What goes awry with the environment instead
of the declaration?
Nick
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Nick Dokos nicholas.do...@hp.com wrote: I think I read the whole thread but I'm still not clear on why the change is needed. What goes awry with the environment instead of the declaration? Never mind - I didn't read the whole thread. Nick
Re: [O] [PATCH] org-e-latex: Tables get correct amount of vertical space
Sebastien Vauban writes:
Hi Myles and Nick,
Nick Dokos wrote:
Myles English mylesengl...@gmail.com wrote:
Sebastien Vauban writes:
Myles English wrote:
This patch replaces every occurence of the \begin{center} environment
with \centering in the file contrib/lisp/org-e-latex.el.
...
- (format \\begin{center}\n%s\\end{center} contents)))
+ (format \\centering\n%s contents)))
Wouldn't you have to replace
\begin{center}
...
\end{center}
by
{\centering
...
}
? That is, add a group around?
I don't think so, at least I have not come across that usage, and it
seems to work without. Do you know different?
If a \centering occurs at top-level in a latex document,
then *everything* after it will be centered: it's a declaration that
remains in force for the current group (which is the rest of the
document if it occurs at top-level).
As confirmed by Nick, \centering just changes the layout for ever... up to the
end of the document.
Why does it work? I guess because there are grouping commands inserted by Org
at other places, that stop the propagation of the centering that far... But
it's dangerous to count on them, IMHO.
\begin{center}...\end{center} is essentially {\centering ...} except
that it also starts a new paragraph.
From within a figure or table environment, that new paragraph shows up
as uneeded vertical space, see the example I give lower down.
It's not clear to me at least, that wholesale replacement is the correct
thing to do: it needs to be looked at on a case-by-case basis I
think.
Nick, I think you are right about this, the modified patch at the end of
this email makes only the replacements that are necessary just to
replace the center environment within table blocks.
The other \begin{center}'s in the org-e-latex.el file may be changed
some other time, if needed.
I think that's a safe replacement (_with_ the grouping), and with something I
forgot about (well seen Nick!): the \par.
I don't think the grouping is necessary because the implied group will
be closed when the table environment is ended. Likewise, I don't think
the \par is necessary.
If you compile this this latex it illustrates the problem, and the
solution:
%__begin example.tex___
\documentclass[11pt]{article}
\usepackage{float}
\begin{document}
Before.---
\begin{table}[htb]
\caption{\label{tab:progress}Summary}
\begin{center}
\begin{tabular}{ll}
A B\\
\end{tabular}
\end{center}
\end{table}
After.
Before.---
\begin{table}[htb]
\caption{\label{tab:progress}Summary}
\centering
\begin{tabular}{ll}
A B\\
\end{tabular}
\end{table}
After.---
\end{document}
%__end example.tex___
See, for example,
https://groups.google.com/forum/?fromgroups=#!topic/fr.comp.text.tex/0s4WfsIfmy8
for the answer of Manuel Pégourié-Gonnard, one reference wrt LaTeX
questions:
My French, and my browser apparently, is not good enough to understand
that thread. Sorry. Please translate if it is still relevant to this
discussion.
replace the environment by {\centering ... \par}
[answered above]
Best regards,
Seb
Myles
From 51686ada4cb2560eb5dfb6b445af547eccff8bff Mon Sep 17 00:00:00 2001
From: Myles English mylesengl...@gmail.com
Date: Tue, 13 Nov 2012 21:04:16 +
Subject: [PATCH] org-e-latex: Tables get correct amount of vertical space
* contrib/lisp/org-e-latex.el: Replaced a set of \begin{center}
and \end{center} environment markers with a \centering
declaration.
Using both \begin{table} and \being{center} environments leads to double
the vertical space around the float, whereas \centering adds none.
TINYCHANGE
---
contrib/lisp/org-e-latex.el | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/contrib/lisp/org-e-latex.el b/contrib/lisp/org-e-latex.el
index 165beb3..5635c6a 100644
--- a/contrib/lisp/org-e-latex.el
+++ b/contrib/lisp/org-e-latex.el
@@ -2302,14 +2302,13 @@ This function assumes TABLE has `org' as its `:type'
attribute.
(concat
(format \\begin{%s}%s\n float-env placement)
(if org-e-latex-table-caption-above caption )))
- (when org-e-latex-tables-centered \\begin{center}\n)
+ (when org-e-latex-tables-centered \\centering\n)
(format \\begin{%s}%s{%s}\n%s\\end{%s}
table-env
(if width (format {%s} width) )
alignment
contents
table-env)
- (when org-e-latex-tables-centered \n\\end{center})
(when float-env
(concat (if org-e-latex-table-caption-above caption)
(format \n\\end{%s} float-env
--
1.8.0
