From c80633fd9f4b289290d036edb66e345407abc5b7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?marcin=20miko=C5=82ajczak?= <git@mkljczk.pl>
Date: Sat, 3 Jun 2023 20:55:04 +0200
Subject: [PATCH] Media modal: Restore navigationHidden functionality
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: marcin mikołajczak <git@mkljczk.pl>
---
 .../ui/components/modals/media-modal.tsx      | 24 ++++++++++++++-----
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/app/soapbox/features/ui/components/modals/media-modal.tsx b/app/soapbox/features/ui/components/modals/media-modal.tsx
index 959c433fd..875506175 100644
--- a/app/soapbox/features/ui/components/modals/media-modal.tsx
+++ b/app/soapbox/features/ui/components/modals/media-modal.tsx
@@ -15,6 +15,7 @@ import PlaceholderStatus from 'soapbox/features/placeholder/components/placehold
 import Thread from 'soapbox/features/status/components/thread';
 import Video from 'soapbox/features/video';
 import { useAppDispatch, useAppSelector } from 'soapbox/hooks';
+import { isUserTouching } from 'soapbox/is-mobile';
 import { makeGetStatus } from 'soapbox/selectors';
 
 import ImageLoader from '../image-loader';
@@ -78,6 +79,8 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
   const handleNextClick = () => setIndex((getIndex() + 1) % media.size);
   const handlePrevClick = () => setIndex((media.size + getIndex() - 1) % media.size);
 
+  const navigationHiddenClassName = navigationHidden ? 'pointer-events-none opacity-0' : '';
+
   const handleKeyDown = (e: KeyboardEvent) => {
     switch (e.key) {
       case 'ArrowLeft':
@@ -101,7 +104,7 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
   const getIndex = () => index !== null ? index : props.index;
 
   const toggleNavigation = () => {
-    setNavigationHidden(!navigationHidden);
+    setNavigationHidden(value => !value && isUserTouching());
   };
 
   const handleStatusClick: React.MouseEventHandler = e => {
@@ -251,7 +254,11 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
           }
           justifyContent='between'
         >
-          <HStack alignItems='center' justifyContent='between' className='flex-[0_0_60px] p-4'>
+          <HStack
+            alignItems='center'
+            justifyContent='between'
+            className={clsx('flex-[0_0_60px] p-4 transition-opacity', navigationHiddenClassName)}
+          >
             <IconButton
               title={intl.formatMessage(messages.close)}
               src={require('@tabler/icons/x.svg')}
@@ -284,9 +291,11 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
           </HStack>
 
           {/* Height based on height of top/bottom bars */}
-          <div className='relative h-[calc(100vh-120px)] w-full grow'>
+          <div
+            className='relative h-[calc(100vh-120px)] w-full grow'
+          >
             {hasMultipleImages && (
-              <div className='absolute inset-y-0 left-5 z-10 flex items-center'>
+              <div className={clsx('absolute inset-y-0 left-5 z-10 flex items-center transition-opacity', navigationHiddenClassName)}>
                 <button
                   tabIndex={0}
                   className='flex h-10 w-10 items-center justify-center rounded-full bg-gray-900 text-white'
@@ -308,7 +317,7 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
             </ReactSwipeableViews>
 
             {hasMultipleImages && (
-              <div className='absolute inset-y-0 right-5 z-10 flex items-center'>
+              <div className={clsx('absolute inset-y-0 right-5 z-10 flex items-center transition-opacity', navigationHiddenClassName)}>
                 <button
                   tabIndex={0}
                   className='flex h-10 w-10 items-center justify-center rounded-full bg-gray-900 text-white'
@@ -322,7 +331,10 @@ const MediaModal: React.FC<IMediaModal> = (props) => {
           </div>
 
           {actualStatus && (
-            <HStack justifyContent='center' className='flex-[0_0_60px]'>
+            <HStack
+              justifyContent='center'
+              className={clsx('flex-[0_0_60px] transition-opacity', navigationHiddenClassName)}
+            >
               <StatusActionBar
                 status={actualStatus}
                 space='md'