Skip to content

Commit 1d6494e

Browse files
committed
Ensure diff sidebar is closed by default
1 parent 7a925a3 commit 1d6494e

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

src/features/sidebar/view/internal/ClientSplitView.tsx

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { Stack, useMediaQuery, useTheme } from "@mui/material";
55
import { isMac, useKeyboardShortcut, SidebarTogglableContext } from "@/common";
66
import { useSidebarOpen } from "../../data";
77
import useDiffbarOpen from "../../data/useDiffbarOpen";
8+
import { useProjectSelection } from "@/features/projects/data";
89
import PrimaryContainer from "./primary/Container";
910
import SecondaryContainer from "./secondary/Container";
1011
import RightContainer from "./tertiary/RightContainer";
@@ -20,6 +21,7 @@ const ClientSplitView = ({
2021
}) => {
2122
const [isSidebarOpen, setSidebarOpen] = useSidebarOpen();
2223
const [isRightSidebarOpen, setRightSidebarOpen] = useDiffbarOpen();
24+
const { specification } = useProjectSelection();
2325
const isSidebarTogglable = useContext(SidebarTogglableContext);
2426
const theme = useTheme();
2527
// Determine if the screen size is small or larger
@@ -30,6 +32,13 @@ const ClientSplitView = ({
3032
setSidebarOpen(true);
3133
}
3234
}, [isSidebarOpen, isSidebarTogglable, setSidebarOpen]);
35+
36+
// Close diff sidebar if no specification is selected
37+
useEffect(() => {
38+
if (!specification && isRightSidebarOpen) {
39+
setRightSidebarOpen(false);
40+
}
41+
}, [specification, isRightSidebarOpen, setRightSidebarOpen]);
3342
useKeyboardShortcut(
3443
(event) => {
3544
const isActionKey = isMac() ? event.metaKey : event.ctrlKey;

0 commit comments

Comments
 (0)