From f8377ea3114df52b7c9509f6c0cb11f0b205dc49 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Vladim=C3=ADr=20Vondru=C5=A1?= <mosra@centrum.cz>
Date: Wed, 1 Nov 2017 23:53:17 +0100
Subject: [PATCH] css: make hiding brand link on landing pages opt-in.

Not all landing pages repeat it, probably just one does that -- the
index one.
---
 css/m-components.css              | 4 ++--
 css/m-dark.compiled.css           | 4 ++--
 css/m-light.compiled.css          | 4 ++--
 doc/css/page-layout.rst           | 8 ++++----
 doc/index.rst                     | 1 +
 doc/pelican/theme.rst             | 4 +++-
 pelican-theme/templates/base.html | 2 +-
 7 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/css/m-components.css b/css/m-components.css
index 50d684ec..f222e19b 100644
--- a/css/m-components.css
+++ b/css/m-components.css
@@ -149,10 +149,10 @@ body> header > nav.m-navbar-jumbo:target {
   background-color: var(--header-background-color);
   opacity: 1;
 }
-body > header > nav.m-navbar-landing a#m-navbar-brand {
+body > header > nav.m-navbar-landing a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: hidden;
 }
-body > header > nav.m-navbar-landing:target a#m-navbar-brand {
+body > header > nav.m-navbar-landing:target a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: visible;
 }
 body > header > nav {
diff --git a/css/m-dark.compiled.css b/css/m-dark.compiled.css
index ab1b172a..71edd6dd 100644
--- a/css/m-dark.compiled.css
+++ b/css/m-dark.compiled.css
@@ -507,10 +507,10 @@ body> header > nav.m-navbar-jumbo:target {
   background-color: #22272e;
   opacity: 1;
 }
-body > header > nav.m-navbar-landing a#m-navbar-brand {
+body > header > nav.m-navbar-landing a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: hidden;
 }
-body > header > nav.m-navbar-landing:target a#m-navbar-brand {
+body > header > nav.m-navbar-landing:target a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: visible;
 }
 body > header > nav {
diff --git a/css/m-light.compiled.css b/css/m-light.compiled.css
index d68d7308..98e7ff04 100644
--- a/css/m-light.compiled.css
+++ b/css/m-light.compiled.css
@@ -507,10 +507,10 @@ body> header > nav.m-navbar-jumbo:target {
   background-color: #ffffff;
   opacity: 1;
 }
-body > header > nav.m-navbar-landing a#m-navbar-brand {
+body > header > nav.m-navbar-landing a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: hidden;
 }
-body > header > nav.m-navbar-landing:target a#m-navbar-brand {
+body > header > nav.m-navbar-landing:target a#m-navbar-brand.m-navbar-brand-hidden {
   visibility: visible;
 }
 body > header > nav {
diff --git a/doc/css/page-layout.rst b/doc/css/page-layout.rst
index 0a1765b9..70db13d8 100644
--- a/doc/css/page-layout.rst
+++ b/doc/css/page-layout.rst
@@ -354,8 +354,9 @@ content "below the fold", *not* the cover image.
 The cover image always spans the whole screen width and goes also under the top
 navbar. In order to make the navbar aware of the image, put a :css:`.m-navbar-landing`
 CSS class on the :html:`<nav>` element --- this makes navbar dimmer with
-transparent background and hides the brand link on the left (with the
-assumption that the landing page includes a bigger version of it). While the
+transparent background. Usually the brand link on the left is superfluous as
+the landing page repeats it in a more prominent place, to hide it put a
+:css:`.m-navbar-brand-hidden` on the :css:`#m-navbar-brand` element. While the
 landing page is designed to catch attention of new users, it shouldn't prevent
 regular visitors from navigating the website --- because of that the top navbar
 is not hidden completely and hovering it will make it more visible. This works
@@ -579,8 +580,7 @@ Similarly to `landing pages <#landing-pages>`_, the cover image of the jumbo
 article always spans the whole screen width and goes below the top navbar. If
 you want the navbar to be semi-transparent, put :css:`.m-navbar-jumbo` on the
 :html:`<nav>` element. Compared to `landing pages <#landing-pages>`_ the navbar
-retains semi-transparent background and the brand link is not hidden, as brand
-name is not expected to be duplicated in article header.
+retains semi-transparent background at all times.
 
 .. note-info::
 
diff --git a/doc/index.rst b/doc/index.rst
index d38e5724..c3751e1a 100644
--- a/doc/index.rst
+++ b/doc/index.rst
@@ -30,6 +30,7 @@ m.css
 :cover: {filename}/static/cover.jpg
 :summary: A no-nonsense, no-JavaScript CSS framework and Pelican theme for
     content-oriented websites
+:navbar_brand_hidden: true
 :landing:
     .. container:: m-row
 
diff --git a/doc/pelican/theme.rst b/doc/pelican/theme.rst
index 7fd9c2e6..cfbb32b8 100644
--- a/doc/pelican/theme.rst
+++ b/doc/pelican/theme.rst
@@ -251,7 +251,8 @@ cover image spanning the whole window width. Put cover image URL into a
 :abbr:`reST <reStructuredText>`-processed content that appears on top of the
 cover image. Contents of the :rst:`:landing:` are put into a
 :html:`<div class="m-container">`, you are expected to fully take care of rows
-and columns in it.
+and columns in it. It's also possible to hide the navbar brand link --- simply
+add a :rst:`:navbar_brand_hidden:` field with non-empty contents.
 
 .. block-warning:: Configuration
 
@@ -275,6 +276,7 @@ destination and URL.
     :save_as: index.html
     :url:
     :cover: {filename}/static/cover.jpg
+    :navbar_brand_hidden: true
     :landing:
         .. container:: m-row
 
diff --git a/pelican-theme/templates/base.html b/pelican-theme/templates/base.html
index 430f06e7..52a421d1 100644
--- a/pelican-theme/templates/base.html
+++ b/pelican-theme/templates/base.html
@@ -23,7 +23,7 @@
 <header><nav id="navigation"{% if page and page.landing and page.cover %} class="m-navbar-landing"{% endif %}{% if article and article.cover %} class="m-navbar-jumbo"{% endif %}>
   <div class="m-container">
     <div class="m-row">
-      <a href="{{ SITEURL }}/" id="m-navbar-brand" class="m-col-t-9 m-col-m-none m-left-m">
+      <a href="{{ SITEURL }}/" id="m-navbar-brand" class="m-col-t-9 m-col-m-none m-left-m{% if page and page.landing and page.cover and page.navbar_brand_hidden %} m-navbar-brand-hidden{% endif %}">
         {%- if SITE_LOGO %}<img src="{{ SITE_LOGO }}" />{% endif -%}
         {{- SITE_LOGO_TEXT -}}
       </a>
-- 
2.30.2