From 99011b0a56cf793e01b2dfa0da161994ff931fe1 Mon Sep 17 00:00:00 2001
From: Fedor Indutny <79877362+indutny-signal@users.noreply.github.com>
Date: Mon, 22 May 2023 22:24:33 +0200
Subject: [PATCH] Fix position of lightbox arrows for a video

---
 stylesheets/components/Lightbox.scss | 11 -----------
 ts/components/Lightbox.tsx           |  4 ----
 2 files changed, 15 deletions(-)

diff --git a/stylesheets/components/Lightbox.scss b/stylesheets/components/Lightbox.scss
index b944fee88257..5e99dc1bc1bb 100644
--- a/stylesheets/components/Lightbox.scss
+++ b/stylesheets/components/Lightbox.scss
@@ -256,17 +256,6 @@
     }
   }
 
-  &__container--has-controls {
-    .Lightbox__nav-next,
-    .Lightbox__nav-prev {
-      // Give some room for <video/>'s controls.
-      $controls-height: 64px;
-
-      padding-top: $controls-height;
-      height: calc(100% - $controls-height);
-    }
-  }
-
   &__controls {
     display: flex;
     gap: 32px;
diff --git a/ts/components/Lightbox.tsx b/ts/components/Lightbox.tsx
index a1f92b8678fe..f4625977d5cb 100644
--- a/ts/components/Lightbox.tsx
+++ b/ts/components/Lightbox.tsx
@@ -484,7 +484,6 @@ export function Lightbox({
   const caption = attachment?.caption;
 
   let content: JSX.Element;
-  let hasControls = false;
   if (!contentType) {
     content = <>{children}</>;
   } else {
@@ -551,8 +550,6 @@ export function Lightbox({
           <source src={objectURL} />
         </video>
       );
-
-      hasControls = true;
     } else if (isUnsupportedImageType || isUnsupportedVideoType) {
       content = (
         <button
@@ -590,7 +587,6 @@ export function Lightbox({
         <div
           className={classNames('Lightbox Lightbox__container', {
             'Lightbox__container--zoom': isZoomed,
-            'Lightbox__container--has-controls': hasControls,
           })}
           onClick={(event: React.MouseEvent<HTMLDivElement>) => {
             event.stopPropagation();