LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Improve resizing screen until it switches to/from Mobile

Browse source
This commit is contained in:
Rafael 2022-04-05 09:59:06 -03:00 committed by Thomas Zarebczan
parent 65b9906086
commit 9f04370516
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
ui/component/fileRenderFloating/view.jsx
View file

@ -112,6 +112,7 @@ export default function FileRenderFloating(props: Props) {
const isMobile = useIsMobile();
const initialMobileState = React.useRef(isMobile);
const initialPlayerHeight = React.useRef();
const resizedBetweenFloating = React.useRef();
@ -170,7 +171,11 @@ export default function FileRenderFloating(props: Props) {
x: rect.x,
};
if (videoAspectRatio && !initialPlayerHeight.current) {
// replace the initial value every time the window is resized if isMobile is true,
// since it could be a portrait -> landscape rotation switch, or if it was a mobile - desktop
// switch, so use the ref to compare the initial state
const resizedEnoughForMobileSwitch = isMobile !== initialMobileState.current;
if (videoAspectRatio && (!initialPlayerHeight.current || isMobile || resizedEnoughForMobileSwitch)) {
const heightForRect = getPossiblePlayerHeight(videoAspectRatio * rect.width, isMobile);
initialPlayerHeight.current = heightForRect;
}
@ -526,7 +531,9 @@ const PlayerGlobalStyles = (props: GlobalStylesProps) => {
const transparentBackground = {
background: videoGreaterThanLandscape && mainFilePlaying && !forceDefaults ? 'transparent !important' : undefined,
};
const maxHeight = { maxHeight: !videoTheaterMode ? 'var(--desktop-portrait-player-max-height)' : undefined };
const maxHeight = {
maxHeight: !videoTheaterMode && !isMobile ? 'var(--desktop-portrait-player-max-height)' : undefined,
};
return (
<Global