Skip to content

Commit 54c4820

Browse files
committed
chore: udpated code to make sure sidebar is collapsed and not toggled
1 parent 759b53e commit 54c4820

File tree

1 file changed

+8
-4
lines changed
  • src/courseware/course/sidebar/sidebars/course-outline

1 file changed

+8
-4
lines changed

src/courseware/course/sidebar/sidebars/course-outline/hooks.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,14 @@ export const useCourseOutlineSidebar = () => {
5454
} = course.entranceExamData || {};
5555
const isActiveEntranceExam = entranceExamEnabled && !entranceExamPassed;
5656

57+
const collapseSidebar = () => {
58+
toggleSidebar(null);
59+
window.sessionStorage.setItem('hideCourseOutlineSidebar', 'true');
60+
};
61+
5762
const handleToggleCollapse = () => {
5863
if (currentSidebar === ID) {
59-
toggleSidebar(null);
60-
window.sessionStorage.setItem('hideCourseOutlineSidebar', 'true');
64+
collapseSidebar();
6165
} else {
6266
toggleSidebar(ID);
6367
window.sessionStorage.removeItem('hideCourseOutlineSidebar');
@@ -111,8 +115,8 @@ export const useCourseOutlineSidebar = () => {
111115
useLayoutEffect(() => {
112116
const handleResize = () => {
113117
// breakpoints.large.maxWidth is 1200px and currently the breakpoint for showing the sidebar
114-
if (isOpen && global.innerWidth < breakpoints.large.maxWidth) {
115-
handleToggleCollapse();
118+
if (currentSidebar === ID && global.innerWidth < breakpoints.large.maxWidth) {
119+
collapseSidebar();
116120
}
117121
};
118122

0 commit comments

Comments
 (0)