From 76fa4576547cb01d4bfecbd7d40cd70dca7b44d2 Mon Sep 17 00:00:00 2001 From: Justin Mayer Date: Sat, 3 Aug 2013 16:17:26 -0700 Subject: [PATCH] Normalize whitespace via lstrip Jinja parameter This adds the lstrip_blocks Jinja parameter and removes unnecessary whitespace from a few notmyidea templates. Note: The lstrip_blocks parameter requires Jinja 2.7+, which has been noted in Pelican's setup.py. Credit for this commit goes entirely to Russ Webber, who has earned my eternal thanks for discovering and applying this useful Jinja parameter. Refs #969 --- pelican/generators.py | 1 + .../basic/a-markdown-powered-article.html | 36 ++--- pelican/tests/output/basic/archives.html | 28 ++-- pelican/tests/output/basic/article-1.html | 36 ++--- pelican/tests/output/basic/article-2.html | 36 ++--- pelican/tests/output/basic/article-3.html | 36 ++--- .../output/basic/author/alexis-metaireau.html | 62 ++++--- pelican/tests/output/basic/categories.html | 28 ++-- pelican/tests/output/basic/category/bar.html | 46 +++--- pelican/tests/output/basic/category/cat1.html | 82 +++++----- pelican/tests/output/basic/category/misc.html | 76 ++++----- pelican/tests/output/basic/category/yeah.html | 46 +++--- .../basic/filename_metadata-example.html | 36 ++--- pelican/tests/output/basic/index.html | 152 ++++++++---------- pelican/tests/output/basic/oh-yeah.html | 38 ++--- .../tests/output/basic/override/index.html | 33 ++-- .../pages/this-is-a-test-hidden-page.html | 33 ++-- .../basic/pages/this-is-a-test-page.html | 33 ++-- .../tests/output/basic/second-article-fr.html | 42 ++--- .../tests/output/basic/second-article.html | 42 ++--- pelican/tests/output/basic/tag/bar.html | 80 +++++---- pelican/tests/output/basic/tag/baz.html | 50 +++--- pelican/tests/output/basic/tag/foo.html | 66 ++++---- pelican/tests/output/basic/tag/foobar.html | 46 +++--- pelican/tests/output/basic/tag/oh.html | 46 +++--- pelican/tests/output/basic/tag/yeah.html | 46 +++--- .../output/basic/this-is-a-super-article.html | 38 ++--- pelican/tests/output/basic/unbelievable.html | 36 ++--- .../custom/a-markdown-powered-article.html | 66 ++++---- pelican/tests/output/custom/archives.html | 54 +++---- pelican/tests/output/custom/article-1.html | 66 ++++---- pelican/tests/output/custom/article-2.html | 66 ++++---- pelican/tests/output/custom/article-3.html | 66 ++++---- .../custom/author/alexis-metaireau.html | 124 +++++++------- .../custom/author/alexis-metaireau2.html | 139 ++++++++-------- .../custom/author/alexis-metaireau3.html | 83 +++++----- pelican/tests/output/custom/categories.html | 54 +++---- pelican/tests/output/custom/category/bar.html | 84 +++++----- .../tests/output/custom/category/cat1.html | 122 +++++++------- .../tests/output/custom/category/misc.html | 114 +++++++------ .../tests/output/custom/category/yeah.html | 78 +++++---- .../output/custom/drafts/a-draft-article.html | 64 ++++---- .../custom/filename_metadata-example.html | 66 ++++---- pelican/tests/output/custom/index.html | 130 +++++++-------- pelican/tests/output/custom/index2.html | 133 +++++++-------- pelican/tests/output/custom/index3.html | 83 +++++----- .../tests/output/custom/jinja2_template.html | 54 +++---- pelican/tests/output/custom/oh-yeah-fr.html | 72 ++++----- pelican/tests/output/custom/oh-yeah.html | 72 ++++----- .../tests/output/custom/override/index.html | 59 ++++--- .../pages/this-is-a-test-hidden-page.html | 59 ++++--- .../custom/pages/this-is-a-test-page.html | 59 ++++--- .../output/custom/second-article-fr.html | 72 ++++----- .../tests/output/custom/second-article.html | 72 ++++----- pelican/tests/output/custom/tag/bar.html | 120 +++++++------- pelican/tests/output/custom/tag/baz.html | 84 +++++----- pelican/tests/output/custom/tag/foo.html | 100 ++++++------ pelican/tests/output/custom/tag/foobar.html | 78 +++++---- pelican/tests/output/custom/tag/oh.html | 84 +++++----- pelican/tests/output/custom/tag/yeah.html | 84 +++++----- .../custom/this-is-a-super-article.html | 66 ++++---- pelican/tests/output/custom/unbelievable.html | 66 ++++---- pelican/themes/notmyidea/templates/index.html | 16 +- pelican/themes/notmyidea/templates/page.html | 4 +- .../notmyidea/templates/translations.html | 2 +- setup.py | 2 +- 66 files changed, 2004 insertions(+), 2143 deletions(-) diff --git a/pelican/generators.py b/pelican/generators.py index 166ff616..13909743 100644 --- a/pelican/generators.py +++ b/pelican/generators.py @@ -56,6 +56,7 @@ class Generator(object): "themes", "simple", "templates")) self.env = Environment( trim_blocks=True, + lstrip_blocks=True, loader=ChoiceLoader([ FileSystemLoader(self._templates_path), simple_loader, # implicit inheritance diff --git a/pelican/tests/output/basic/a-markdown-powered-article.html b/pelican/tests/output/basic/a-markdown-powered-article.html index 29c6d0d6..6f854ae4 100644 --- a/pelican/tests/output/basic/a-markdown-powered-article.html +++ b/pelican/tests/output/basic/a-markdown-powered-article.html @@ -4,8 +4,8 @@ A markdown powered article - - + + @@ -15,46 +15,46 @@ -
+
-
+