From 157f5db69058e52878ec4ad8ee745694e356637e Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Wed, 6 Feb 2019 21:00:30 -0800
Subject: [PATCH] fix: fix zoomable videos (#947)

fixes #946
---
 .../dialog/components/MediaDialog.html        | 41 +++++++++++--------
 1 file changed, 25 insertions(+), 16 deletions(-)

diff --git a/src/routes/_components/dialog/components/MediaDialog.html b/src/routes/_components/dialog/components/MediaDialog.html
index bc8d046..06668a1 100644
--- a/src/routes/_components/dialog/components/MediaDialog.html
+++ b/src/routes/_components/dialog/components/MediaDialog.html
@@ -11,20 +11,26 @@
         <div class="media-scroll-item">
           <div class="media-scroll-item-inner">
             <div class="media-scroll-item-inner-inner">
-                <PinchZoomable className='media-pinch-zoom' disabled={!pinchZoomMode} >
+                {#if canPinchZoom}
+                  <PinchZoomable className='media-pinch-zoom' disabled={!pinchZoomMode} >
+                    <MediaInDialog {media} />
+                  </PinchZoomable>
+                {:else}
                   <MediaInDialog {media} />
-                </PinchZoomable>
+                {/if}
             </div>
           </div>
         </div>
       {/each}
     </div>
     <div class="media-controls-outside">
-      <IconButton
-        className="media-control-button media-control-button-dummy-spacer"
-        href="#fa-search"
-        label=""
-      />
+      {#if canPinchZoom}
+        <IconButton
+          className="media-control-button media-control-button-dummy-spacer"
+          href="#fa-search"
+          label=""
+        />
+      {/if}
       {#if dots.length > 1}
         <div class="media-controls">
             <IconButton
@@ -54,14 +60,16 @@
             />
         </div>
       {/if}
-      <IconButton
-        className="media-control-button"
-        pressable={true}
-        pressed={pinchZoomMode}
-        label={pinchZoomMode ? 'Disable pinch-zoom mode' : 'Enable pinch-zoom mode'}
-        href="#fa-search"
-        on:click="togglePinchZoomMode()"
-      />
+      {#if canPinchZoom}
+        <IconButton
+          className="media-control-button"
+          pressable={true}
+          pressed={pinchZoomMode}
+          label={pinchZoomMode ? 'Disable pinch-zoom mode' : 'Enable pinch-zoom mode'}
+          href="#fa-search"
+          on:click="togglePinchZoomMode()"
+        />
+      {/if}
     </div>
   </div>
 </ModalDialog>
@@ -206,7 +214,8 @@
     }),
     computed: {
       length: ({ mediaItems }) => mediaItems.length,
-      dots: ({ length }) => times(length, i => ({ i }))
+      dots: ({ length }) => times(length, i => ({ i })),
+      canPinchZoom: ({ mediaItems }) => !mediaItems.some(media => media.type === 'video')
     },
     components: {
       ModalDialog,