#44 no default cover (#45)

support for HOME_COVER and HOME_COLOR

.gitignore

@@ -1,3 +1,4 @@
 *.pyc
 *.log
 output
+tests/.cache/

README.adoc

@@ -81,14 +81,20 @@ You can set cover images for your blog, article, page, tag, category and author.
 
 The is the cover image for your site main index.html.
 
-To set blog cover, set the property `HEADER_COVER` in
+To set blog cover, set the property `HOME_COVER` in
 `pelicanconf.py`:
 
 [source,python]
 ----
-HEADER_COVER = '/assets/images/blog_cover.png'
+HOME_COVER = '/assets/images/blog_cover.png'
 ----
 
+[NOTE]
+====
+HEADER_COVER property is deprecated. Work around will be use HOME_COVER and use cover in individual articles.
+This property will be used if there is no cover image set to an article, page, tag, category, author.
+====
+
 [[article-cover]]
 ==== Article Cover
 
@@ -167,17 +173,22 @@ For more refer link:#author-bio[author] .
 === Header Color
 
 To define a simple header background color, set the property
-`HEADER_COLOR` in `pelicanconf.py`:
+`HOME_COLOR` in `pelicanconf.py`:
 
 [source,python]
 ----
-HEADER_COLOR = 'black'
+HOME_COLOR = 'black'
 ----
 
 you can use any valid css color. This will be used if there is no cover
 image set in link:#articles[article] level and site level.
 
-This property will be used if there is no cover image set to an article, page, tag, category, author.
+
+[NOTE]
+====
+HEADER_COLOR property is deprecated. Work around will be use HOME_COLOR and use color in individual articles.
+This property will be used if there is no HEADER_COVER and cover image set to an article, page, tag, category, author.
+====
 
 [[social-urls]]
 === Social URLs

templates/archives.html

@@ -15,7 +15,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -36,8 +36,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/article.html

@@ -63,7 +63,7 @@
 
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="post-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -102,8 +102,6 @@
             <div class="post-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
             <div class="post-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-            <div class="post-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/author.html

@@ -46,7 +46,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -66,14 +66,12 @@
             <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
             <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-            <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>
 
     {% if AUTHORS_BIO and author.name.lower() in AUTHORS_BIO %}
-    <section id="blog-author" class="has-cover" >
+    <section id="blog-author" {% if selected_cover or selected_color %}class="has-cover"{% endif %} >
         <div class="inner">
             <aside class="post-author">
                 {% if selected_avatar %}

templates/authors.html

@@ -16,7 +16,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -37,8 +37,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/categories.html

@@ -15,7 +15,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -36,8 +36,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/category.html

@@ -23,7 +23,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -45,8 +45,6 @@
             <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
             <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-            <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/index.html

@@ -12,14 +12,22 @@
   {% else %}
     {% set selected_cover = SITEURL+"/"+HEADER_COVER %}
   {% endif %}
+{% elif HOME_COVER %}
+  {% if HOME_COVER|lower|truncate(4, True, '') == "http" %}
+    {% set selected_cover = HOME_COVER %}
+  {% else %}
+    {% set selected_cover = SITEURL+"/"+HOME_COVER %}
+  {% endif %}
 {% elif HEADER_COLOR %}
   {% set selected_color = HEADER_COLOR %}
+{% elif HOME_COLOR %}
+  {% set selected_color = HOME_COLOR %}
 {% endif %}
 
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -40,8 +48,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/page.html

@@ -30,7 +30,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="post-header" class="has-cover">
+    <header id="post-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -51,8 +51,6 @@
             <div class="post-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
             <div class="post-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-            <div class="post-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/period_archives.html

@@ -2,10 +2,20 @@
 
 {% block title %}{{ SITENAME }} - Archives for {{ period | reverse | join (' ') }}{% endblock %}
 
+{% if HEADER_COVER %}
+  {% if HEADER_COVER|lower|truncate(4, True, '') == "http" %}
+    {% set selected_cover = HEADER_COVER %}
+  {% else %}
+    {% set selected_cover = SITEURL+"/"+HEADER_COVER %}
+  {% endif %}
+{% elif HEADER_COLOR %}
+  {% set selected_color = HEADER_COLOR %}
+{% endif %}
+
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -22,15 +32,13 @@
           </span>
         </nav>
         <h1 class="post-title">Archives for {{ period | reverse | join (' ') }}</h1>
-        {% if HEADER_COVER %}
+        {% if selected_cover %}
-            <div class="blog-cover cover" style="background-image: url('{{ HEADER_COVER }}')">            
+          <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
-        {% elif HEADER_COLOR %}
+        {% elif selected_color %}
-            <div class="post-cover cover" style="background-color: {{ HEADER_COLOR }}">
+          <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
+        {% endif %}
-            <div class="post-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
-        {% endif %}        
       </div>
-    </header>    
+    </header>
 {% endblock header %}
 
 {% block content %}

templates/tag.html

@@ -23,7 +23,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -45,8 +45,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

templates/tags.html

@@ -15,7 +15,7 @@
 {% block header %}
     <!-- Page Header -->
     <!-- Set your background image for this header on the line below. -->
-    <header id="blog-header" class="has-cover">
+    <header id="blog-header" {% if selected_cover or selected_color %}class="has-cover"{% endif %}>
       <div class="inner">
         <nav id="navigation">
           {% if SITE_LOGO %}
@@ -36,8 +36,6 @@
           <div class="blog-cover cover" style="background-image: url('{{ selected_cover }}')">
         {% elif selected_color %}
           <div class="blog-cover cover" style="background-color: {{ selected_color }}">
-        {% else %}
-          <div class="blog-cover cover" style="background-image: url('{{ SITEURL }}/{{ THEME_STATIC_DIR }}/images/post-bg.jpg')">
         {% endif %}
       </div>
     </header>

tests/default_conf.py

@@ -4,6 +4,7 @@ from __future__ import unicode_literals
 
 AUTHOR = u'Zutrinken'
 SITENAME = u'Attila Demo'
+SITESUBTITLE = u'Blog description here.'
 SITEURL = ''
 
 PATH = 'content'

tests/test_coverimages.py

@@ -59,9 +59,9 @@ class ArticleCoverImageTest(unittest.TestCase, BaseTest):
   def test_article_theme_cover(self):
     rstPath="content/article_without_cover.rst"
     result, soup = self.gen_article_and_html_from_rst(rstPath)
-    selected = soup.find(name="div", attrs={"class": "post-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_article_header_color(self):
     self.settings['HEADER_COLOR']='blue'
@@ -120,9 +120,9 @@ class PageCoverImageTest(unittest.TestCase, BaseTest):
   def test_page_theme_cover(self):
     rstPath="content/pages/page_without_cover_image.rst"
     result, soup = self.gen_page_and_html_from_rst(rstPath)
-    selected = soup.find(name="div", attrs={"class": "post-cover cover"})
+    selected = soup.find(id="post-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_page_header_color(self):
     self.settings['HEADER_COLOR']='blue'
@@ -156,9 +156,9 @@ class TagCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_footag_theme_cover(self):
     result, soup = self.gen_tag_and_html_from_name("footag")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_footag_cover(self):
     tagName = "footag"
@@ -204,9 +204,9 @@ class TagCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_bartag_theme_cover(self):
     result, soup = self.gen_tag_and_html_from_name("bartag")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_bartag_cover(self):
     tagName = "bartag"
@@ -261,9 +261,9 @@ class CategoryCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_foo_theme_cover(self):
     result, soup = self.gen_category_and_html_from_name("foo")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_foo_cover(self):
     categoryName = "foo"
@@ -309,9 +309,9 @@ class CategoryCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_bar_theme_cover(self):
     result, soup = self.gen_category_and_html_from_name("bar")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_bar_cover(self):
     categoryName = "bar"
@@ -365,9 +365,9 @@ class AuthorCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_arul_theme_cover(self):
     result, soup = self.gen_author_and_html_from_name("arul")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_arul_cover(self):
     authorName = "arul"
@@ -417,9 +417,9 @@ class AuthorCoverImageTest(unittest.TestCase, BaseTest):
 
   def test_raj_theme_cover(self):
     result, soup = self.gen_author_and_html_from_name("raj")
-    selected = soup.find(name="div", attrs={"class": "blog-cover cover"})
+    selected = soup.find(id="blog-header")
     # Assertion
-    self.assertTrue("/theme/images/post-bg.jpg" in selected["style"])
+    self.assertTrue("class" not in selected)
 
   def test_raj_cover(self):
     authorName = "raj"