Skip to content

Commit dfd4987

Browse files
committed
app: Replace local/session docs toggle with state.
Having this persist across sessions is not necessary and a bit overly complicated when we add more sideview states such as camera (which should not stay on by default).
1 parent 7e4a9b9 commit dfd4987

File tree

2 files changed

+10
-44
lines changed

2 files changed

+10
-44
lines changed

src/app/App.tsx

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@ import './app.scss';
66
import { Button, Classes, Spinner } from '@blueprintjs/core';
77
import { Manual } from '@blueprintjs/icons';
88
import React, { useEffect, useState } from 'react';
9+
10+
type SideView = 'off' | 'docs';
911
import SplitterLayout from 'react-splitter-layout';
1012
import { useLocalStorage, useTernaryDarkMode } from 'usehooks-ts';
1113
import Activities from '../activities/Activities';
1214
import DfuWindowsDriverInstallDialog from '../firmware/dfuWindowsDriverInstallDialog/DfuWindowsDriverInstallDialog';
1315
import { InstallPybricksDialog } from '../firmware/installPybricksDialog/InstallPybricksDialog';
1416
import RestoreOfficialDialog from '../firmware/restoreOfficialDialog/RestoreOfficialDialog';
15-
import { useSettingIsShowDocsEnabled } from '../settings/hooks';
1617
import SponsorDialog from '../sponsor/SponsorDialog';
1718
import StatusBar from '../status-bar/StatusBar';
1819
import Toolbar from '../toolbar/Toolbar';
@@ -72,8 +73,7 @@ const Docs: React.FunctionComponent = () => {
7273
const App: React.FunctionComponent = () => {
7374
const i18n = useI18n();
7475
const { isDarkMode } = useTernaryDarkMode();
75-
const { isSettingShowDocsEnabled, toggleIsSettingShowDocsEnabled } =
76-
useSettingIsShowDocsEnabled();
76+
const [sideView, setSideView] = useState<SideView>('off');
7777
const [isDragging, setIsDragging] = useState(false);
7878

7979
const [docsSplit, setDocsSplit] = useLocalStorage('app-docs-split', 30);
@@ -118,7 +118,7 @@ const App: React.FunctionComponent = () => {
118118
<div className="pb-app-main" style={{ position: 'relative' }}>
119119
<SplitterLayout
120120
customClassName={
121-
isSettingShowDocsEnabled ? 'pb-show-docs' : 'pb-hide-docs'
121+
sideView === 'docs' ? 'pb-show-docs' : 'pb-hide-docs'
122122
}
123123
onDragStart={(): void => setIsDragging(true)}
124124
onDragEnd={(): void => setIsDragging(false)}
@@ -148,11 +148,15 @@ const App: React.FunctionComponent = () => {
148148
large
149149
icon={<Manual />}
150150
title={
151-
isSettingShowDocsEnabled
151+
sideView === 'docs'
152152
? i18n.translate('docs.hide')
153153
: i18n.translate('docs.show')
154154
}
155-
onClick={toggleIsSettingShowDocsEnabled}
155+
onClick={() =>
156+
setSideView(
157+
sideView === 'docs' ? 'off' : 'docs',
158+
)
159+
}
156160
/>
157161
</main>
158162
<aside

src/settings/hooks.ts

Lines changed: 0 additions & 38 deletions
This file was deleted.

0 commit comments

Comments
 (0)