Media Viewer: Fix muting hidden slides (#1690)
This commit is contained in:
parent
33c6fe5df5
commit
fa755aad9c
@ -6,7 +6,7 @@ import {
|
|||||||
} from '../../api/types';
|
} from '../../api/types';
|
||||||
import { MediaViewerOrigin } from '../../types';
|
import { MediaViewerOrigin } from '../../types';
|
||||||
|
|
||||||
import { IS_SINGLE_COLUMN_LAYOUT } from '../../util/environment';
|
import { IS_SINGLE_COLUMN_LAYOUT, IS_TOUCH_ENV } from '../../util/environment';
|
||||||
import useBlurSync from '../../hooks/useBlurSync';
|
import useBlurSync from '../../hooks/useBlurSync';
|
||||||
import useMedia from '../../hooks/useMedia';
|
import useMedia from '../../hooks/useMedia';
|
||||||
import useMediaWithLoadProgress from '../../hooks/useMediaWithLoadProgress';
|
import useMediaWithLoadProgress from '../../hooks/useMediaWithLoadProgress';
|
||||||
@ -187,7 +187,11 @@ const MediaViewerContent: FC<OwnProps & StateProps> = (props) => {
|
|||||||
message && calculateMediaViewerDimensions(dimensions!, hasFooter),
|
message && calculateMediaViewerDimensions(dimensions!, hasFooter),
|
||||||
!IS_SINGLE_COLUMN_LAYOUT && !isProtected,
|
!IS_SINGLE_COLUMN_LAYOUT && !isProtected,
|
||||||
)}
|
)}
|
||||||
{isVideo && (isActive ? (
|
{isVideo && ((!isActive && IS_TOUCH_ENV) ? renderVideoPreview(
|
||||||
|
bestImageData,
|
||||||
|
message && calculateMediaViewerDimensions(dimensions!, hasFooter, false),
|
||||||
|
!IS_SINGLE_COLUMN_LAYOUT && !isProtected,
|
||||||
|
) : (
|
||||||
<VideoPlayer
|
<VideoPlayer
|
||||||
key={messageId}
|
key={messageId}
|
||||||
url={localBlobUrl || fullMediaBlobUrl}
|
url={localBlobUrl || fullMediaBlobUrl}
|
||||||
@ -200,10 +204,6 @@ const MediaViewerContent: FC<OwnProps & StateProps> = (props) => {
|
|||||||
noPlay={!isActive}
|
noPlay={!isActive}
|
||||||
onClose={onClose}
|
onClose={onClose}
|
||||||
/>
|
/>
|
||||||
) : renderVideoPreview(
|
|
||||||
bestImageData,
|
|
||||||
message && calculateMediaViewerDimensions(dimensions!, hasFooter, false),
|
|
||||||
!IS_SINGLE_COLUMN_LAYOUT && !isProtected,
|
|
||||||
))}
|
))}
|
||||||
{textParts && (
|
{textParts && (
|
||||||
<MediaViewerFooter
|
<MediaViewerFooter
|
||||||
|
|||||||
@ -524,7 +524,7 @@ const MediaViewerSlides: FC<OwnProps> = ({
|
|||||||
/* eslint-disable-next-line react/jsx-props-no-spreading */
|
/* eslint-disable-next-line react/jsx-props-no-spreading */
|
||||||
{...rest}
|
{...rest}
|
||||||
messageId={activeMessageId}
|
messageId={activeMessageId}
|
||||||
isActive={isActiveRef.current}
|
isActive={isActive && isActiveRef.current}
|
||||||
isFooterHidden={isFooterHidden || isZoomed || scale !== 1}
|
isFooterHidden={isFooterHidden || isZoomed || scale !== 1}
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user