Use individual <a> for each tag/category/etc in post metadata

Fixes #85

assets/css/bobastyle.css

@@ -312,6 +312,20 @@ footer p {
   margin: 0;
 }
 
+#skip-to-main {
+  position: absolute;
+  padding: 8px;
+  background-color: var(--text-0);
+  color: var(--background-0);
+  transform: translateY(-100%);
+}
+#skip-to-main:focus {
+  transform: translateY(0%);
+}
+#main-content:target {
+  animation: none; /* prevent it from turning yellow */
+}
+
 /* }}} */
 
 /* Top bar formatting {{{ */
@@ -326,7 +340,7 @@ footer p {
    * or it will need its hover background to be chopped off on the left */
 }
 
-.header {
+.brand {
   flex-grow: 1;
   margin: 8px var(--page-margin);
   font-size: 1.25em;
@@ -334,7 +348,7 @@ footer p {
   align-items: center;
 }
 
-.header a {
+.brand a {
   color: inherit;
   font-weight: bold;
 }
@@ -360,22 +374,6 @@ footer p {
 
 /* }}} */
 
-/* Skip to main content {{{ */
-#skip-to-main {
-  position: absolute;
-  padding: 8px;
-  background-color: var(--text-0);
-  color: var(--background-0);
-  transform: translateY(-100%);
-}
-#skip-to-main:focus {
-  transform: translateY(0%);
-}
-#main-content:target {
-  animation: none; /* prevent it from turning yellow */
-}
-/* }}} */
-
 /* Breadcrumb navigation {{{ */
 
 .breadcrumb {
@@ -508,6 +506,14 @@ footer p {
   margin-top: 10px;
 }
 
+:is(
+  .page-metadata-section:is(.categories, .tags, .series),
+  .page-metadata-item.authors
+) > a:not(:last-child):after {
+  color: var(--text-0);
+  content: ",";
+}
+
 .post-media {
   margin-top: 15px;
 }
@@ -634,10 +640,6 @@ footer p {
   margin-bottom: 5px;
 }
 
-.view-section {
-  margin-top: 5px;
-}
-
 /* }}} */
 
 
@@ -705,19 +707,6 @@ footer p {
 }
 */
 
-/* See also formating */
-.see-also {
-  background-color: var(--background-1);
-  color: var(--text-1);
-  padding: 8px 12px;
-  border-radius: 8px;
-  margin: 10px 0;
-}
-
-.see-also p {
-  margin: 0;
-}
-
 /* Previous and next page buttons {{{ */
 
 .prevnext {
@@ -768,6 +757,45 @@ footer p {
 
 /* }}} */
 
+/* Related posts section {{{ */
+
+/* set the margins on the contents instead of the parent .related-posts
+ * so that the posts in the list clip off the edge of the screen,
+ * which makes a more obvious indicator that it's scrollable
+ */
+
+.related-posts > hr,
+.related-posts > h1 {
+  margin: var(--page-margin);
+}
+
+.related-posts .page {
+  min-width: 300px;
+  max-width: 300px;
+
+}
+
+.related-posts .page:first-child {
+  margin-left: var(--page-margin);
+}
+.related-posts .page:last-child {
+  margin-right: var(--page-margin);
+}
+
+.related-posts .page-list {
+  display: flex;
+  flex-direction: row;
+  gap: 20px;
+  overflow-x: auto;
+  margin-bottom: var(--page-margin);
+}
+
+/* }}} */
+
+/* }}} */
+
+/* Content components (can appear both in single and list pages) {{{ */
+
 /* Code & code blocks {{{ */
 .code-block {
   display: flex;
@@ -784,10 +812,13 @@ footer p {
   padding: 4px 8px;
   border-top-left-radius: 8px;
   border-top-right-radius: 8px;
+  height: 2em;
+}
+.code-block > .code-header > * {
+  margin: auto 0;
 }
 .code-block > .code-header > .code-type {
   border-top-left-radius: 8px;
-  margin: auto 0;
 }
 /* TODO: make the code copy button prettier */
 .code-block > .code-header > .code-copy-button {
@@ -881,42 +912,21 @@ aside.quote {
 
 /* }}} */
 
-/* Related posts section {{{ */
+/* See also formating */
-
+.see-also {
-/* set the margins on the contents instead of the parent .related-posts
+  background-color: var(--background-1);
- * so that the posts in the list clip off the edge of the screen,
+  color: var(--text-1);
- * which makes a more obvious indicator that it's scrollable
+  padding: 8px 12px;
- */
+  border-radius: 8px;
-
+  margin: 10px 0;
-.related-posts > hr,
-.related-posts > h1 {
-  margin: var(--page-margin);
 }
 
-.related-posts .page {
+.see-also p {
-  min-width: 300px;
+  margin: 0;
-  max-width: 300px;
-
-}
-
-.related-posts .page:first-child {
-  margin-left: var(--page-margin);
-}
-.related-posts .page:last-child {
-  margin-right: var(--page-margin);
-}
-
-.related-posts .page-list {
-  display: flex;
-  flex-direction: row;
-  gap: 20px;
-  overflow-x: auto;
-  margin-bottom: var(--page-margin);
 }
 
 /* }}} */
 
-/* }}} */
 
 
 /* @media specializations {{{ */

assets/css/comments.css

@@ -1,3 +1,14 @@
+.comments > h2 {
+    margin-bottom: 0;
+}
+.comments .email-address-notice {
+    margin: 0;
+    font-size: smaller;
+}
+.comments .enable-javascript-notice {
+    font-style: italic;
+}
+
 /* Isso styling */
 h4.isso-thread-heading {
     color: var(--text-0);

assets/css/noscript.css

@@ -0,0 +1,4 @@
+.page-meta-comments-counter,
+.code-copy-button {
+  display: none;
+}

layouts/_partials/comments.html

@@ -1,15 +1,20 @@
+{{/* put comments CSS here instead of in <head> so that it doesn't get
+   * loaded unnecessarily on regular pages
+   */}}
 {{ with resources.Get "css/comments.css" | fingerprint "sha512" }}
     <link rel="stylesheet" type="text/css" href="{{ .Permalink }}" integrity="{{ .Data.Integrity }}" crossorigin="anonymous">
 {{ end }}
 
-<h2 style="margin-bottom: 0;">Comments</h2>
+<h2>Comments</h2>
-<p style="margin: 0; font-size: smaller;">
+<p class="email-address-notice">
     If you provide an email address, you can enable notifications for
     replies to your comment. It will not be shown publicly.
 </p>
 
 <noscript>
-    <i>Enable JavaScript to see the comment section.</i>
+    <p class="enable-javascript-notice">
+        Enable JavaScript to see the comment section.
+    </p>
 </noscript>
 
 <section id="isso-thread" data-title="{{ .Title }}"></section>

layouts/_partials/head.html

@@ -6,6 +6,12 @@
     {{ with resources.Get "css/bobastyle.css" | fingerprint "sha512" }}
         <link rel="stylesheet" type="text/css" href="{{ .Permalink }}" integrity="{{ .Data.Integrity }}" crossorigin="anonymous">
     {{ end }}
+    {{ with resources.Get "css/noscript.css" | fingerprint "sha512" }}
+        <noscript>
+            <link rel="stylesheet" type="text/css" href="{{ .Permalink }}" integrity="{{ .Data.Integrity }}" crossorigin="anonymous">
+        </noscript>
+    {{ end }}
+
     {{ with resources.Get "css/syntax.css" | fingerprint "sha512" }}
         <link rel="stylesheet" type="text/css" href="{{ .Permalink }}" integrity="{{ .Data.Integrity }}" crossorigin="anonymous" media="print" onload="this.media='all'">
     {{ end }}

layouts/_partials/page-metadata/item/authors.html

@@ -1,16 +1,9 @@
-{{ with .Params.authors }}
+{{ with .GetTerms "authors" }}
-    <span class="page-metadata-item" data-pagefind-meta="authors" aria-label="{{ i18n "aria_post_meta_authors" }}">
+    <span class="page-metadata-item authors" aria-label="{{ i18n "aria_post_meta_authors" }}">
         {{ partial "icon.html" "user-circle" }}
 
-        {{ if index $.Site.Taxonomies "authors" }}
+        {{ range . }}
-            {{ $authors := slice }}
+            <a href="{{ .Permalink }}" data-pagefind-filter="author">{{ .LinkTitle }}</a>
-            {{ range . }}
-                {{ $url := (printf "authors/%s" (. | anchorize)) | absLangURL }}
-                {{ $authors = $authors | append (printf `<a href="%s">%s</a>` $url .) }}
-            {{ end }}
-            {{ delimit $authors ", " | safeHTML }}
-        {{ else }}
-            {{ delimit . ", " }}
         {{ end }}
     </span>
 {{ end }}

layouts/_partials/page-metadata/item/categories.html

@@ -4,8 +4,10 @@
         {{ $categories = $categories | append (printf `<a href="%s">%s</a>` .Permalink .LinkTitle) }}
     {{ end }}
 
-    <div class="page-metadata-section categories" data-pagefind-meta="categories" aria-label="{{ i18n "aria_post_meta_categories" }}">
+    <div class="page-metadata-section categories" aria-label="{{ i18n "aria_post_meta_categories" }}">
         {{ partial "icon.html" "folder" }}
-        {{ delimit $categories ", " | safeHTML }}
+        {{ range . }}
+            <a href="{{ .Permalink }}" data-pagefind-filter="category">{{ .LinkTitle }}</a>
+        {{ end }}
     </div>
 {{ end }}

layouts/_partials/page-metadata/item/comments-counter.html

@@ -1,4 +1,4 @@
-<span class="page-metadata-item" aria-label="comments counter">
+<span class="page-metadata-item page-meta-comments-counter" aria-label="comments counter">
     {{ partial "icon.html" "message" }}
     {{/* TODO: figure out if there's a nicer way to generate this URL
        * the current issue is that we can't use RelRef, since content view means that we don't

layouts/_partials/page-metadata/item/series.html

@@ -1,11 +1,8 @@
 {{ with (.GetTerms "series") }}
-    {{ $series := slice }}
+    <div class="page-metadata-section series" aria-label="{{ i18n "aria_post_meta_series" }}">
-    {{ range . }}
-        {{ $series = $series | append (printf `<a href="%s">%s</a>` .Permalink .LinkTitle) }}
-    {{ end }}
-
-    <div class="page-metadata-section series" data-pagefind-meta="series" aria-label="{{ i18n "aria_post_meta_series" }}">
         {{ partial "icon.html" "files" }}
-        {{ delimit $series ", " | safeHTML }}
+        {{ range . }}
+            <a href="{{ .Permalink }}" data-pagefind-filter="series">{{ .LinkTitle }}</a>
+        {{ end }}
     </div>
 {{ end }}

layouts/_partials/page-metadata/item/tags.html

@@ -1,11 +1,10 @@
 {{ with (.GetTerms "tags") }}
     {{ $tags := slice }}
-    {{ range . }}
-        {{ $tags = $tags | append (printf `<a href="%s">%s</a>` .Permalink .LinkTitle) }}
-    {{ end }}
 
-    <div class="page-metadata-section tags" data-pagefind-meta="tags" aria-label="{{ i18n "aria_post_meta_tags" }}">
+    <div class="page-metadata-section tags" aria-label="{{ i18n "aria_post_meta_tags" }}">
         {{ partial "icon.html" "tag" }}
-        {{ delimit $tags ", " | safeHTML }}
+        {{ range . }}
+            <a href="{{ .Permalink }}" data-pagefind-filter="tag">{{ .LinkTitle }}</a>
+        {{ end }}
     </div>
 {{ end }}

layouts/_partials/seo-tags/opengraph.html

@@ -3,16 +3,6 @@
 <meta property="og:type" content="{{ if .IsPage }}article{{ else }}website{{ end }}">
 <meta property="og:url" content="{{ .Permalink }}">
 
-{{ $images := $.Resources.ByType "image" }}
-{{ $featured := $images.GetMatch "*feature*" }}
-{{ if not $featured }}
-    {{ $featured = $images.GetMatch "{*cover*,*thumbnail*}" }}
-{{ end }}
-
-{{ with $featured }}
-    <meta property="og:image" content="{{ $featured.Permalink }}">
-{{ end }}
-
 {{ with .Section }}
     <meta property="article:section" content="{{ . }}">
 {{ end }}
@@ -37,12 +27,38 @@
     <meta property="og:site_name" content="{{ . }}">
 {{ end }}
 
+{{ $images := $.Resources.ByType "image" }}
+{{ $featured := $images.GetMatch "*feature*" }}
+{{ if not $featured }}
+    {{ $featured = $images.GetMatch "{*cover*,*thumbnail*}" }}
+{{ end }}
+
+{{ with $featured }}
+    <meta property="og:image" content="{{ $featured.Permalink }}">
+    {{ with .Params.alt }}
+        <meta property="og:image:alt" content="{{ . }}">
+    {{ end }}
+    {{ with .MediaType }}
+        <meta property="og:image:type" content="{{ . }}">
+    {{ end }}
+    {{ with .Width }}
+        <meta property="og:image:width" content="{{ . }}">
+    {{ end }}
+    {{ with .Height }}
+        <meta property="og:image:height" content="{{ . }}">
+    {{ end }}
+{{ end }}
+
 {{ $videos := $.Resources.ByType "video" }}
 {{ $featured_video := $videos.GetMatch "*feature*" }}
 {{ if not $featured_video }}
     {{ $featured_video = $videos.GetMatch "{*cover*,*thumbnail*}" }}
 {{ end }}
 
+
 {{ with $featured_video }}
     <meta property="og:video" content="{{ $featured_video.Permalink | absURL }}">
+    {{ with .MediaType }}
+        <meta property="og:video:type" content="{{ . }}">
+    {{ end }}
 {{ end }}

layouts/_partials/seo-tags/twitter-cards.html

@@ -31,6 +31,8 @@
    */}}
 {{ with .Site.Params.social.twitter }}
     <meta name="twitter:site" content="@{{ . }}">
+{{ else }}
+    {{ errorf "a twitter username is required for twitter cards to work" }}
 {{ end }}
 {{ with .GetTerms "authors" }}
     {{ range . }}

layouts/_partials/top.html

@@ -1,9 +1,9 @@
-<div class="top">
+<header class="top">
-    <header class="header">
+    <div class="brand">
         <a href="{{ .Site.Home.Permalink | absLangURL }}">
             {{ .Site.Title | markdownify }}
         </a>
-    </header>
+    </div>
 
     <nav class="navbar" aria-label="{{ i18n "aria_navbar" }}">
         {{ with .Site.Home }}
@@ -21,4 +21,4 @@
                href="{{ .URL }}">{{ .Name }}</a>
         {{ end }}
     </nav>
-</div>
+</header>