From 2913de47acb01111c12d7ad1c6df80efff060d4a Mon Sep 17 00:00:00 2001 From: AhtishamShahid Date: Tue, 6 Dec 2022 11:45:59 +0500 Subject: [PATCH] fix: updated sidebar logic --- src/discussions/data/hooks.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/discussions/data/hooks.js b/src/discussions/data/hooks.js index 775d039b..046d8fba 100644 --- a/src/discussions/data/hooks.js +++ b/src/discussions/data/hooks.js @@ -25,12 +25,13 @@ import { selectIsCourseStaff, selectLearnersTabEnabled, selectModerationSettings, - selectPostThreadCount, + selectPostThreadCount, selectThreadCallStatus, selectUserHasModerationPrivileges, selectUserIsGroupTa, selectUserIsStaff, } from './selectors'; import { fetchCourseConfig } from './thunks'; +import { selectLearnerPages } from '../learners/data/selectors'; export function useTotalTopicThreadCount() { const topics = useSelector(selectTopics); @@ -51,11 +52,15 @@ export const useSidebarVisible = () => { const isViewingTopics = useRouteMatch(Routes.TOPICS.PATH); const isViewingLearners = useRouteMatch(Routes.LEARNERS.PATH); + if (isViewingTopics && totalThreads < 1) { + return false; + } + if (isFiltered) { return true; } - if ((isViewingTopics || isViewingLearners) && totalThreads > 0) { + if (isViewingTopics || isViewingLearners) { return true; }