From fb39759dd33211fb20b08df48e0ffcfcca5d1426 Mon Sep 17 00:00:00 2001
From: DokterKaj <54882101+DokterKaj@users.noreply.github.com>
Date: Mon, 21 Oct 2024 23:03:48 +0800
Subject: [PATCH] display gallery as flexbox + add borders to media

---
 static/style.css     | 55 +++++++++++++++++++++++++++++++++++++++++---
 templates/utils.html |  7 ++++--
 2 files changed, 57 insertions(+), 5 deletions(-)

diff --git a/static/style.css b/static/style.css
index a9d893a..eae84b1 100644
--- a/static/style.css
+++ b/static/style.css
@@ -223,8 +223,8 @@ nav #redlib {
     vertical-align: -2px;
 }
 
-figcaption {
-    margin-top: 5px;
+figcaption p {
+    margin: 5px 0;
     text-align: center;
 }
 
@@ -1112,6 +1112,13 @@ a.search_subreddit:hover {
     overflow: hidden;
 }
 
+.post_media_content:not(:has(.post_media_video)),
+.post_media_video,
+.gallery {
+    border: var(--panel-border);
+    border-radius: 5px;
+}
+
 .post_media_video {
     min-width: 100px;
     max-width: 100px;
@@ -1169,9 +1176,51 @@ a.search_subreddit:hover {
     vertical-align: bottom;
 }
 
+.gallery {
+    display: flex;
+    flex-flow: row nowrap;
+    overflow-x: auto;
+    scroll-snap-type: x mandatory;
+    align-items: center;
+    background: var(--background);
+    transition: background 0.2s;
+}
+
+.post:hover .gallery {
+    background: var(--post);
+}
+
+.gallery figure {
+    flex: 1 0 100%;
+    scroll-snap-align: center;
+    scroll-snap-stop: always;
+    margin: 0;
+}
+
+.gallery_length {
+    position: sticky;
+    flex-shrink: 0;
+    align-self: start;
+    right: 2%;
+    top: 2%;
+    width: 6em;
+    padding: 10px 5px;
+    margin-left: calc(-6em - 5px*2);
+    text-align: center;
+    background-color: rgba(0, 0, 0, 0.8);
+    border-radius: 5px;
+    transition: opacity 0.2s;
+}
+
+.gallery_length:focus {
+    opacity: 0;
+}
+
 .gallery img {
+    display: block;
+    margin: auto;
     max-width: 100%;
-    vertical-align: bottom;
+    border-radius: 5px;
 }
 
 .gallery .outbound_url {
diff --git a/templates/utils.html b/templates/utils.html
index e0b3589..12e213f 100644
--- a/templates/utils.html
+++ b/templates/utils.html
@@ -136,17 +136,20 @@
 	{% endif %}
 	{% else if post.post_type == "gallery" %}
 	<div class="gallery">
-	{% for image in post.gallery -%}
+		{% for image in post.gallery -%}
 		<figure>
 			<a href="{{ image.url }}" ><img loading="lazy" alt="Gallery image" src="{{ image.url }}"/></a>
 			<figcaption>
+				{% if !image.caption.is_empty() %}
 				<p>{{ image.caption }}</p>
+				{% endif %}
 				{% if image.outbound_url.len() > 0 %}
 				<p><a class="outbound_url" href="{{ image.outbound_url }}" rel="nofollow">{{ image.outbound_url }}</a>
 				{% endif %}
 			</figcaption>
 		</figure>
-	{%- endfor %}
+		{%- endfor %}
+		<div class="gallery_length" tabindex="-1">gallery<br>({{ post.len() }} images)</div>
 	</div>
 	{% else if post.post_type == "link" %}
 	<a id="post_url" href="{{ post.media.url }}" rel="nofollow">{{ post.media.url }}</a>