chore: remove unused code (#574)

Author: just-boris

src/app-layout/index.tsx

@@ -10,7 +10,6 @@ import { applyDefaults } from './defaults';
 import { AppLayoutProps } from './interfaces';
 import { Notifications } from './notifications';
 import { MobileToolbar } from './mobile-toolbar';
-import { SplitPanelWrapper } from './split-panel-wrapper';
 import { useFocusControl } from './utils/use-focus-control';
 import useWindowWidth from './utils/use-window-width';
 import useContentHeight from './utils/use-content-height';
@@ -21,7 +20,11 @@ import { AppLayoutContext } from '../internal/context/app-layout-context';
 import { useContainerQuery } from '../internal/hooks/container-queries';
 import { useStableEventHandler } from '../internal/hooks/use-stable-event-handler';
 import { applyDisplayName } from '../internal/utils/apply-display-name';
-import { SplitPanelContextProps, SplitPanelLastInteraction } from '../internal/context/split-panel-context';
+import {
+  SplitPanelContext,
+  SplitPanelContextProps,
+  SplitPanelLastInteraction,
+} from '../internal/context/split-panel-context';
 import {
   CONSTRAINED_MAIN_PANEL_MIN_HEIGHT,
   CONSTRAINED_PAGE_HEIGHT,
@@ -340,6 +343,9 @@ const OldAppLayout = React.forwardRef(
       onPreferencesChange: onSplitPanelPreferencesSet,
       reportSize: setSplitPanelReportedSize,
     };
+    const splitPanelWrapped = splitPanel && (
+      <SplitPanelContext.Provider value={splitPanelContext}>{splitPanel}</SplitPanelContext.Provider>
+    );
 
     const contentWrapperProps: ContentWrapperProps = {
       navigationPadding: navigationHide || !!navigationOpen,
@@ -551,13 +557,11 @@ const OldAppLayout = React.forwardRef(
                   </div>
                 </ContentWrapper>
               </div>
-              {finalSplitPanePosition === 'bottom' && (
-                <SplitPanelWrapper context={splitPanelContext}>{splitPanel}</SplitPanelWrapper>
-              )}
+              {finalSplitPanePosition === 'bottom' && splitPanelWrapped}
             </main>
 
             <ToolsAndSplitPanel
-              splitPanel={finalSplitPanePosition === 'side' ? splitPanel : undefined}
+              splitPanel={finalSplitPanePosition === 'side' ? splitPanelWrapped : undefined}
               ariaLabels={ariaLabels}
               closedDrawerWidth={closedDrawerWidth}
               contentHeightStyle={contentHeightStyle}
@@ -570,7 +574,6 @@ const OldAppLayout = React.forwardRef(
               isMotionEnabled={isMotionEnabled}
               onToolsToggle={onToolsToggle}
               panelHeightStyle={panelHeightStyle}
-              splitPanelContext={splitPanelContext}
               splitPanelOpen={splitPanelOpenOnTheSide}
               splitPanelReportedSize={splitPanelReportedSize}
               toggleRefs={toolsRefs}

src/app-layout/split-panel-wrapper.tsx

@@ -14,37 +14,3 @@
   top: awsui.$size-vertical-panel-icon-offset;
   z-index: 1;
 }
-
-.circular-toggle {
-  cursor: pointer;
-  border-radius: 50%;
-  border: none;
-  width: awsui.$space-layout-toggle-diameter;
-  height: awsui.$space-layout-toggle-diameter;
-  padding: 0 awsui.$space-s;
-  background: awsui.$color-background-layout-toggle-default;
-  color: awsui.$color-text-layout-toggle;
-  box-shadow: awsui.$shadow-panel-toggle;
-  &:focus {
-    outline: none;
-  }
-  @include focus-visible.when-visible {
-    @include styles.focus-highlight(3px);
-  }
-  &:hover {
-    background: awsui.$color-background-layout-toggle-hover;
-  }
-  &:active {
-    background: awsui.$color-background-layout-toggle-active;
-  }
-  &-selected {
-    background: awsui.$color-background-layout-toggle-selected-default;
-    color: awsui.$color-text-layout-toggle-selected;
-    &:hover {
-      background: awsui.$color-background-layout-toggle-selected-hover;
-    }
-    &:active {
-      background: awsui.$color-background-layout-toggle-selected-active;
-    }
-  }
-}

src/app-layout/toggles/circular-button.tsx

@@ -4,11 +4,9 @@ import clsx from 'clsx';
 import React from 'react';
 import { DesktopDrawerProps, Drawer } from './drawer';
 import { AppLayoutProps } from './interfaces';
-import { SplitPanelWrapper } from './split-panel-wrapper';
 import useContentHeight from './utils/use-content-height';
 import styles from './styles.css.js';
 import testutilStyles from './test-classes/styles.css.js';
-import { SplitPanelContextProps } from '../internal/context/split-panel-context';
 
 interface ToolsAndSplitPanelProps {
   toolsHide: boolean;
@@ -28,8 +26,7 @@ interface ToolsAndSplitPanelProps {
   contentHeightStyle: ReturnType<typeof useContentHeight>['contentHeightStyle'];
 
   tools: React.ReactNode;
-  splitPanel?: React.ReactNode;
-  splitPanelContext: SplitPanelContextProps;
+  splitPanel: React.ReactNode;
 
   ariaLabels: AppLayoutProps['ariaLabels'];
 
@@ -54,7 +51,6 @@ export function ToolsAndSplitPanel({
   onToolsToggle,
   panelHeightStyle,
   splitPanel,
-  splitPanelContext,
   toggleRefs,
   onLoseToolsFocus,
   tools,
@@ -81,7 +77,7 @@ export function ToolsAndSplitPanel({
             ...(isMobile ? { top: headerHeight, bottom: footerHeight } : panelHeightStyle),
           }}
         >
-          {splitPanel && <SplitPanelWrapper context={splitPanelContext}>{splitPanel}</SplitPanelWrapper>}
+          {splitPanel}
           {!toolsHide && (
             <Drawer
               type="tools"

src/app-layout/toggles/styles.scss

@@ -20,7 +20,6 @@ export interface SplitPanelContextProps {
     closedNav: boolean;
     closedTools: boolean;
   };
-  isCopy?: boolean;
   isOpen?: boolean;
   isMobile: boolean;
   isForcedPosition: boolean;
@@ -47,7 +46,6 @@ export const SplitPanelContext = createContext<SplitPanelContextProps>({
   getMaxWidth: () => 0,
   getMaxHeight: () => 0,
   getHeader: () => null,
-  isCopy: false,
   isOpen: true,
   isMobile: false,
   isForcedPosition: false,

src/app-layout/tools-and-split-panel.tsx

@@ -51,7 +51,6 @@ export default function SplitPanel({
     bottomOffset,
     rightOffset,
     contentWidthStyles,
-    isCopy,
     isOpen,
     isForcedPosition,
     splitPanelRef,
@@ -255,7 +254,7 @@ export default function SplitPanel({
    * is still needed for the early return to prevent execution
    * of the following code.
    */
-  if (isRefresh && (!isOpen || isCopy) && position === 'side') {
+  if (isRefresh && !isOpen && position === 'side') {
     return <></>;
   }