From ada6b9f699254feef9d1e4dd65a963e890ae1b4c Mon Sep 17 00:00:00 2001
From: sgenoud <steve@sgenoud.com>
Date: Thu, 22 Aug 2019 07:31:14 +0200
Subject: [PATCH] fix: Fix the blurhash alignment and general style (#1418)

* Fix the blurhash alignment issue

* Fix the video blurhash style
---
 src/routes/_components/status/Media.html      | 22 ++--------------
 .../status/StatusMediaAttachments.html        | 25 ++++++++++++++++---
 2 files changed, 23 insertions(+), 24 deletions(-)

diff --git a/src/routes/_components/status/Media.html b/src/routes/_components/status/Media.html
index 15dffbf6..12a49efd 100644
--- a/src/routes/_components/status/Media.html
+++ b/src/routes/_components/status/Media.html
@@ -1,21 +1,4 @@
-{#if type === 'video' || type === 'audio'}
-  {#if blurhash}
-    {#if type === 'video'}
-      <div>
-        <LazyImage
-          alt={description}
-          title={description}
-          src={previewUrl}
-          fallback={oneTransparentPixel}
-          blurhash={blurhash}
-          width={inlineWidth}
-          height={inlineHeight}
-          background="var(--loading-bg)"
-          {focus}
-        />
-      </div>
-    {/if}
-  {:else}
+{#if (type === 'video' && !blurhash) || type === 'audio'}
     <button id={elementId}
             type="button"
             class="play-video-button focus-after {$largeInlineMedia ? '' : 'fixed-size'} {type === 'audio' ? 'play-audio-button' : ''}"
@@ -36,7 +19,6 @@
         />
       {/if}
     </button>
-  {/if}
 {:else}
   <button id={elementId}
           type="button"
@@ -54,7 +36,7 @@
       height={inlineHeight}
       {focus}
     />
-  {:elseif type === 'gifv'}
+  {:elseif type === 'gifv' && !blurhash}
     <NonAutoplayGifv
       class={noNativeWidthHeight ? 'no-native-width-height' : ''}
       label="Animated GIF: {description}"
diff --git a/src/routes/_components/status/StatusMediaAttachments.html b/src/routes/_components/status/StatusMediaAttachments.html
index 2f7740e4..a7a86097 100644
--- a/src/routes/_components/status/StatusMediaAttachments.html
+++ b/src/routes/_components/status/StatusMediaAttachments.html
@@ -54,13 +54,12 @@
   }
 
   .status-sensitive-media-container.grouped-images .status-sensitive-inner-div {
-    position: absolute;
+    position: relative;
     width: 100%;
   }
 
   .status-sensitive-media-container.grouped-images {
     grid-area: media-grp;
-    padding-bottom: 58%;
   }
 
   .status-sensitive-media-button {
@@ -87,6 +86,7 @@
 
   .status-sensitive-media-shown .status-sensitive-media-button.status-sensitive-media-button-transparent {
     top: 5px;
+    left: 5px;
   }
 
   .status-sensitive-media-hidden .status-sensitive-media-button {
@@ -153,6 +153,7 @@
 
   .status-sensitive-media-hidden .svg-wrapper.svg-wrapper-transparent {
     top: 5px;
+    left: 5px;
   }
 
   .status-sensitive-media-container.grouped-images .svg-wrapper {
@@ -200,9 +201,11 @@
     },
     store: () => store,
     computed: {
-      computedClass: ({ sensitiveShown, $largeInlineMedia }) => classname(
+      computedClass: ({ sensitiveShown, oddCols, twoCols, $largeInlineMedia }) => classname(
         'status-sensitive-media-container',
         sensitiveShown ? 'status-sensitive-media-shown' : 'status-sensitive-media-hidden',
+        oddCols && 'odd-cols',
+        twoCols && 'two-cols',
         $largeInlineMedia ? 'not-grouped-images' : 'grouped-images'
       ),
       mediaAttachments: ({ originalStatus }) => originalStatus.media_attachments,
@@ -233,7 +236,21 @@
       sensitiveMediaIconClass: ({ canUseBlurhash }) => classname(
         'status-sensitive-media-svg',
         canUseBlurhash && 'status-sensitive-media-svg-transparent'
-      )
+      ),
+      nCols:
+        ({ mediaAttachments, $largeInlineMedia }) => {
+          return (!$largeInlineMedia && mediaAttachments.length > 1) ? 2 : 1
+        },
+      oddCols:
+        ({ mediaAttachments }) => {
+          return (mediaAttachments.length > 1 && (mediaAttachments.length % 2))
+        },
+
+      twoCols:
+        ({ mediaAttachments }) => {
+          return (mediaAttachments.length === 2)
+        }
+
     },
     methods: {
       toggleSensitiveMedia () {