Merge pull request #95 from RSE-Sheffield/bugfix_ampersand

Bugfix: Avoid double html escaping of lesson title.

DESCRIPTION

@@ -19,6 +19,11 @@ Authors@R: c(
            family = "Michonneau",
            role = c("ctb"),
            email = "[email protected]"),
+    person(given = "Robert",
+           family = "Chisholm",
+           role = c("ctb"),
+           email = "[email protected]",
+           comment = c(ORCID = "0000-0003-3379-9042")),
     person()
     )
 Description: This package does nothing but contain template HTML CSS and JS files.

inst/pkgdown/templates/head.html

@@ -1,5 +1,5 @@
     <meta charset="utf-8">
-    <title>{{#site}}{{title}}{{/site}}{{#pagetitle}}: {{&pagetitle}}{{/pagetitle}}</title>
+    <title>{{#site}}{{&title}}{{/site}}{{#pagetitle}}: {{&pagetitle}}{{/pagetitle}}</title>
     <meta name="viewport" content="width=device-width, initial-scale=1">
     <link rel="stylesheet" type="text/css" href="{{#site}}{{root}}{{/site}}assets/styles.css">
     <script src="{{#site}}{{root}}{{/site}}assets/scripts.js" type="text/javascript"></script>

inst/pkgdown/templates/header.html

@@ -108,7 +108,7 @@
       <img class="small-logo" alt="{{#yaml}}{{carpentry_name}}{{/yaml}}" src="{{#site}}{{root}}{{/site}}assets/images/{{#yaml}}{{carpentry_icon}}-logo-sm.svg{{/yaml}}">
     </div>
     <div class="lesson-title-md">
-      {{#site}}{{title}}{{/site}}
+      {{#site}}{{&title}}{{/site}}
     </div>
     <div class="search-icon-sm">
       <!-- TODO: do not show until we have search
@@ -119,7 +119,7 @@
       <ul class="navbar-nav me-auto mb-2 mb-lg-0">
         <li class="nav-item">
           <span class="lesson-title">
-            {{#site}}{{title}}{{/site}}
+            {{#site}}{{&title}}{{/site}}
           </span>
         </li>
         {{#overview}}
@@ -181,7 +181,7 @@
 </nav>
 
 <div class="col-md-12 mobile-title">
-  {{#site}}{{title}}{{/site}}
+  {{#site}}{{&title}}{{/site}}
 </div>
 
 {{^overview}}