From 664e24cf86f0cd543714a4ffe596abbb7a902efd Mon Sep 17 00:00:00 2001 From: jfbu Date: Mon, 13 Jun 2016 10:30:48 +0200 Subject: [PATCH 1/2] Fix #2671: image directive may lead to inconsistent spacing in pdf --- sphinx/texinputs/sphinx.sty | 37 ++++++++++++++++++++----------------- sphinx/writers/latex.py | 13 +++++++++---- tests/test_build_latex.py | 18 +++++++++--------- 3 files changed, 38 insertions(+), 30 deletions(-) diff --git a/sphinx/texinputs/sphinx.sty b/sphinx/texinputs/sphinx.sty index f75a8d194..6c3365c1e 100644 --- a/sphinx/texinputs/sphinx.sty +++ b/sphinx/texinputs/sphinx.sty @@ -710,26 +710,29 @@ \raggedright} {\end{list}} -% Redefine includgraphics for avoiding images larger than the screen size -% If the size is not specified. +% Re-define \includegraphics to resize images larger than the line width +% if the size is not specified. +% Warning: future version of Sphinx will not modify original \includegraphics, +% Below custom code will be direct definition of \sphinxincludegraphics, with +% \py@Oldincludegraphics replaced by direct use of original \includegraphics. \let\py@Oldincludegraphics\includegraphics - -\newbox\image@box% -\newdimen\image@width% -\renewcommand\includegraphics[2][\@empty]{% - \ifx#1\@empty% - \setbox\image@box=\hbox{\py@Oldincludegraphics{#2}}% - \image@width\wd\image@box% - \ifdim \image@width>\linewidth% - \setbox\image@box=\hbox{\py@Oldincludegraphics[width=\linewidth]{#2}}% - \box\image@box% - \else% - \py@Oldincludegraphics{#2}% - \fi% - \else% +\newbox\spx@image@box +\renewcommand*{\includegraphics}[2][\@empty]{% + \ifx\@empty #1% attention, #1 could be bb.., bad if first after \ifx + \setbox\spx@image@box=\hbox{\py@Oldincludegraphics{#2}}% + \ifdim \wd\spx@image@box>\linewidth + \py@Oldincludegraphics[width=\linewidth]{#2}% + \else + \leavevmode\box\spx@image@box + \fi + \else \py@Oldincludegraphics[#1]{#2}% - \fi% + \fi } +% Writer will put \sphinxincludegraphics in LaTeX source, and with this, +% documents which used their own modified \includegraphics will compile +% as before. But see warning above. +\newcommand*{\sphinxincludegraphics}{\includegraphics} % to make pdf with correct encoded bookmarks in Japanese % this should precede the hyperref package diff --git a/sphinx/writers/latex.py b/sphinx/writers/latex.py index 524e035a7..d7bd3297c 100644 --- a/sphinx/writers/latex.py +++ b/sphinx/writers/latex.py @@ -408,7 +408,8 @@ class LaTeXTranslator(nodes.NodeVisitor): self.elements['date'] = format_date(builder.config.today_fmt or _('%b %d, %Y'), language=builder.config.language) if builder.config.latex_logo: - self.elements['logo'] = '\\includegraphics{%s}\\par' % \ + # no need for \\noindent here, used in flushright + self.elements['logo'] = '\\sphinxincludegraphics{%s}\\par' % \ path.basename(builder.config.latex_logo) # setup babel self.babel = ExtBabel(builder.config.language) @@ -1407,8 +1408,12 @@ class LaTeXTranslator(nodes.NodeVisitor): except KeyError: pass if not is_inline: - pre.append('\n') - post.append('\n') + if isinstance(node.parent, nodes.section): + pre.append('\n\\noindent') + post.append('\n') + else: + pre.append('\n') + post.append('\n') pre.reverse() if node['uri'] in self.builder.images: uri = self.builder.images[node['uri']] @@ -1425,7 +1430,7 @@ class LaTeXTranslator(nodes.NodeVisitor): if include_graphics_options: options = '[%s]' % ','.join(include_graphics_options) base, ext = path.splitext(uri) - self.body.append('\\includegraphics%s{{%s}%s}' % (options, base, ext)) + self.body.append('\\sphinxincludegraphics%s{{%s}%s}' % (options, base, ext)) self.body.extend(post) def depart_image(self, node): diff --git a/tests/test_build_latex.py b/tests/test_build_latex.py index 503f41ef1..f7479f545 100644 --- a/tests/test_build_latex.py +++ b/tests/test_build_latex.py @@ -113,19 +113,19 @@ def test_writer(app, status, warning): result = (app.outdir / 'SphinxTests.tex').text(encoding='utf8') assert ('\\begin{sphinxfigure-in-table}\n\\centering\n\\capstart\n' - '\\includegraphics{{img}.png}\n\\sphinxfigcaption' + '\\sphinxincludegraphics{{img}.png}\n\\sphinxfigcaption' '{figure in table}\\label{markup:id7}\\end{sphinxfigure-in-table}' in result) assert ('\\begin{wrapfigure}{r}{0pt}\n\\centering\n' - '\\includegraphics{{rimg}.png}\n\\caption{figure with align option}' + '\\sphinxincludegraphics{{rimg}.png}\n\\caption{figure with align option}' '\\label{markup:id8}\\end{wrapfigure}' in result) assert ('\\begin{wrapfigure}{r}{0.500\\linewidth}\n\\centering\n' - '\\includegraphics{{rimg}.png}\n\\caption{figure with align \\& figwidth option}' + '\\sphinxincludegraphics{{rimg}.png}\n\\caption{figure with align \\& figwidth option}' '\\label{markup:id9}\\end{wrapfigure}' in result) assert ('\\begin{wrapfigure}{r}{3cm}\n\\centering\n' - '\\includegraphics[width=3cm]{{rimg}.png}\n' + '\\sphinxincludegraphics[width=3cm]{{rimg}.png}\n' '\\caption{figure with align \\& width option}' '\\label{markup:id10}\\end{wrapfigure}' in result) @@ -574,12 +574,12 @@ def test_image_in_section(app, status, warning): print(result) print(status.getvalue()) print(warning.getvalue()) - assert ('\chapter[Test section]' - '{\includegraphics[width=15pt,height=15pt]{{pic}.png} Test section}' + assert ('\\chapter[Test section]' + '{\\sphinxincludegraphics[width=15pt,height=15pt]{{pic}.png} Test section}' in result) - assert ('\chapter[Other {[}blah{]} section]{Other {[}blah{]} ' - '\includegraphics[width=15pt,height=15pt]{{pic}.png} section}' in result) - assert ('\chapter{Another section}' in result) + assert ('\\chapter[Other {[}blah{]} section]{Other {[}blah{]} ' + '\\sphinxincludegraphics[width=15pt,height=15pt]{{pic}.png} section}' in result) + assert ('\\chapter{Another section}' in result) @with_app(buildername='latex', confoverrides={'latex_logo': 'notfound.jpg'}) From beeeb56c27233d1deb956abe1d4cf00990ba6284 Mon Sep 17 00:00:00 2001 From: jfbu Date: Sun, 19 Jun 2016 11:02:39 +0200 Subject: [PATCH 2/2] latex ``\noindent`` for non-inline images, not only if top level --- sphinx/writers/latex.py | 10 +++------- tests/test_build_latex.py | 21 ++++++++++++--------- 2 files changed, 15 insertions(+), 16 deletions(-) diff --git a/sphinx/writers/latex.py b/sphinx/writers/latex.py index d7bd3297c..229e62902 100644 --- a/sphinx/writers/latex.py +++ b/sphinx/writers/latex.py @@ -1408,12 +1408,8 @@ class LaTeXTranslator(nodes.NodeVisitor): except KeyError: pass if not is_inline: - if isinstance(node.parent, nodes.section): - pre.append('\n\\noindent') - post.append('\n') - else: - pre.append('\n') - post.append('\n') + pre.append('\n\\noindent') + post.append('\n') pre.reverse() if node['uri'] in self.builder.images: uri = self.builder.images[node['uri']] @@ -1456,7 +1452,7 @@ class LaTeXTranslator(nodes.NodeVisitor): self.body.append('\\begin{sphinxfigure-in-table}\n\\centering\n') if any(isinstance(child, nodes.caption) for child in node): self.body.append('\\capstart') - self.context.append(ids + '\\end{sphinxfigure-in-table}\n') + self.context.append(ids + '\\end{sphinxfigure-in-table}\\relax\n') elif node.get('align', '') in ('left', 'right'): if 'width' in node: length = width_to_latex_length(node['width']) diff --git a/tests/test_build_latex.py b/tests/test_build_latex.py index f7479f545..4ef83165f 100644 --- a/tests/test_build_latex.py +++ b/tests/test_build_latex.py @@ -113,21 +113,24 @@ def test_writer(app, status, warning): result = (app.outdir / 'SphinxTests.tex').text(encoding='utf8') assert ('\\begin{sphinxfigure-in-table}\n\\centering\n\\capstart\n' - '\\sphinxincludegraphics{{img}.png}\n\\sphinxfigcaption' - '{figure in table}\\label{markup:id7}\\end{sphinxfigure-in-table}' in result) + '\\noindent\\sphinxincludegraphics{{img}.png}\n' + '\\sphinxfigcaption{figure in table}\\label{markup:id7}' + '\\end{sphinxfigure-in-table}\\relax' in result) assert ('\\begin{wrapfigure}{r}{0pt}\n\\centering\n' - '\\sphinxincludegraphics{{rimg}.png}\n\\caption{figure with align option}' - '\\label{markup:id8}\\end{wrapfigure}' in result) + '\\noindent\\sphinxincludegraphics{{rimg}.png}\n' + '\\caption{figure with align option}\\label{markup:id8}' + '\\end{wrapfigure}' in result) assert ('\\begin{wrapfigure}{r}{0.500\\linewidth}\n\\centering\n' - '\\sphinxincludegraphics{{rimg}.png}\n\\caption{figure with align \\& figwidth option}' - '\\label{markup:id9}\\end{wrapfigure}' in result) + '\\noindent\\sphinxincludegraphics{{rimg}.png}\n' + '\\caption{figure with align \\& figwidth option}\\label{markup:id9}' + '\\end{wrapfigure}' in result) assert ('\\begin{wrapfigure}{r}{3cm}\n\\centering\n' - '\\sphinxincludegraphics[width=3cm]{{rimg}.png}\n' - '\\caption{figure with align \\& width option}' - '\\label{markup:id10}\\end{wrapfigure}' in result) + '\\noindent\\sphinxincludegraphics[width=3cm]{{rimg}.png}\n' + '\\caption{figure with align \\& width option}\\label{markup:id10}' + '\\end{wrapfigure}' in result) @with_app(buildername='latex', freshenv=True, # use freshenv to check warnings