From 5c273ce8c27f87a0d3cdd0d42e316c928f4244ae Mon Sep 17 00:00:00 2001
From: "max.mehl" <max.mehl@fsfe.org>
Date: Thu, 27 Feb 2020 11:49:55 +0100
Subject: [PATCH] do not create an ul for every link

---
 .../layouts/shortcodes/links.html             | 30 +++++++++----------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/themes/hugo-sustain/layouts/shortcodes/links.html b/themes/hugo-sustain/layouts/shortcodes/links.html
index bb058e0..6331a75 100644
--- a/themes/hugo-sustain/layouts/shortcodes/links.html
+++ b/themes/hugo-sustain/layouts/shortcodes/links.html
@@ -3,20 +3,20 @@
 {{- if $type -}}
   {{- $data = where $data "type" $type -}}
 {{ end }}
+<ul>
 {{- range sort $data "date" "desc" }}
-  <ul>
-    <li>{{.desc}} – 
-    {{- $len := (len .links) -}} <!-- amount of elements under .links -->
-      {{- range $index, $element := .links -}} <!-- range over links, but keep index -->
-        {{ with .href }}<a href="{{.}}" target="_blank">{{ end }}{{.text}}{{ with .href }}</a>{{end}}
-        {{- if ne (add $index 1) $len -}}, {{- end -}} <!-- only append comma if not last element (index starts with 0) -->
-      {{- end -}}
-      {{- if or (.video) (.audio) (.slides) }},{{ end -}}
-      {{ with .video }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Video Recording"><i class="fas fa-video" aria-hidden="true" title="Video Recording"></i></a>{{ end }}
-      {{- with .audio }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Audio Recording"><i class="fas fa-microphone" aria-hidden="true" title="Audio Recording"></i></a>{{ end }}
-      {{ with .slides }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Slides"><i class="fas fa-desktop" aria-hidden="true" title="Slides"></i></a>{{ end -}}
-      {{- with .date }} ({{.}}){{ end }} 
-      {{ range .tags }}<span class="label label-success">{{ . }}</span>&nbsp;{{ end }}
-    </li>
-  </ul>
+  <li>{{.desc}}&nbsp;–&nbsp;
+  {{- $len := (len .links) -}} <!-- amount of elements under .links -->
+    {{- range $index, $element := .links -}} <!-- range over links, but keep index -->
+      {{ with .href }}<a href="{{.}}" target="_blank">{{ end }}{{.text}}{{ with .href }}</a>{{end}}
+      {{- if ne (add $index 1) $len -}}, {{- end -}} <!-- only append comma if not last element (index starts with 0) -->
+    {{- end -}}
+    {{- if or (.video) (.audio) (.slides) }},{{ end -}}
+    {{ with .video }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Video Recording"><i class="fas fa-video" aria-hidden="true" title="Video Recording"></i></a>{{ end }}
+    {{- with .audio }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Audio Recording"><i class="fas fa-microphone" aria-hidden="true" title="Audio Recording"></i></a>{{ end }}
+    {{ with .slides }}&nbsp;<a href="{{ . }}" target="_blank" aria-label="Slides"><i class="fas fa-desktop" aria-hidden="true" title="Slides"></i></a>{{ end -}}
+    {{- with .date }} ({{.}}){{ end }}
+    {{ range .tags }}<span class="label label-success">{{ . }}</span>&nbsp;{{ end }}
+  </li>
 {{ end }}
+</ul>