Andal.LND/frontend-app-discussions
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: replace bodyheight with paragon windowSize (#327)

Browse Source
This commit is contained in:
Awais Ansari
2022-10-14 15:42:28 +05:00
committed by GitHub
parent 401d38ce05
commit 8ef83c5ae8
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/discussions/discussions-home/DiscussionSidebar.jsx
View File

@@ -7,6 +7,8 @@ import {
Redirect, Route, Switch, useLocation,
} from 'react-router';
import { useWindowSize } from '@edx/paragon';
import { RequestStatus, Routes } from '../../data/constants';
import {
useContainerSize, useIsOnDesktop, useIsOnXLDesktop, useShowLearnersTab,
@@ -24,13 +26,14 @@ export default function DiscussionSidebar({ displaySidebar, postActionBarRef })
const redirectToLearnersTab = useShowLearnersTab();
const sidebarRef = useRef(null);
const postActionBarHeight = useContainerSize(postActionBarRef);
const { height: windowHeight } = useWindowSize();
useEffect(() => {
if (sidebarRef && postActionBarHeight) {
if (isOnDesktop) {
sidebarRef.current.style.maxHeight = `${document.body.offsetHeight - postActionBarHeight}px`;
sidebarRef.current.style.maxHeight = `${windowHeight - postActionBarHeight}px`;
}
sidebarRef.current.style.minHeight = `${document.body.offsetHeight - postActionBarHeight}px`;
sidebarRef.current.style.minHeight = `${windowHeight - postActionBarHeight}px`;
sidebarRef.current.style.top = `${postActionBarHeight}px`;
}
}, [sidebarRef, postActionBarHeight]);