Merge pull request #17906 from JonathanKolnik/jono/ch-1793-the-sideba…

…r-is-still-appearing-in UI: Fix canvas as initialActive for fullscreen mode in mobile
storybookjs · Apr 7, 2022 · 6ade743 · 6ade743
2 parents e973eea + 19de11a
commit 6ade743
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/ui/src/components/layout/mobile.stories.tsx b/lib/ui/src/components/layout/mobile.stories.tsx
@@ -33,6 +33,13 @@ export const InitialAddons = ({ props }: { props: MobileProps }) => (
   <Mobile {...props} options={{ ...props.options, initialActive: ActiveTabs.ADDONS }} />
 );
 
+export const Fullscreen = ({ props }: { props: MobileProps }) => (
+  <Mobile
+    {...props}
+    options={{ ...props.options, initialActive: ActiveTabs.SIDEBAR, isFullscreen: true }}
+  />
+);
+
 export const DocsOnly = ({ props }: { props: MobileProps }) => <Mobile {...props} docsOnly />;
 
 export const Page = ({ props }: { props: MobileProps }) => (

diff --git a/lib/ui/src/components/layout/mobile.tsx b/lib/ui/src/components/layout/mobile.tsx
@@ -156,7 +156,7 @@ class Mobile extends Component<MobileProps, MobileState> {
 
     const { options } = props;
     this.state = {
-      active: options.initialActive || SIDEBAR,
+      active: options.isFullscreen ? CANVAS : options.initialActive || SIDEBAR,
     };
   }