chore: Do not hide scrollbars in dev pages in Mac OS (#3168)

cloudscape-design · Jan 14, 2025 · f9aeff8 · f9aeff8
1 parent d46a1d8
commit f9aeff8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 19 deletions.
diff --git a/pages/app/index.tsx b/pages/app/index.tsx
@@ -16,7 +16,7 @@ import StrictModeWrapper from './components/strict-mode-wrapper';
 // import font-size reset and Ember font
 import '@cloudscape-design/global-styles/index.css';
 // screenshot test overrides
-import styles from './styles.scss';
+import './styles.scss';
 
 interface GlobalFlags {
   appLayoutWidget?: boolean;
@@ -57,7 +57,6 @@ function App() {
   // Also, AppLayout pages should resemble the ConsoleNav 2.0 styles
   const isAppLayout = isAppLayoutPage(pageId);
   const ContentTag = isAppLayout ? 'div' : 'main';
-  const isMacOS = navigator.userAgent.toLowerCase().indexOf('macintosh') > -1;
 
   useEffect(() => {
     applyMode(mode ?? null);
@@ -71,14 +70,6 @@ function App() {
     disableMotion(motionDisabled);
   }, [motionDisabled]);
 
-  useEffect(() => {
-    if (isMacOS) {
-      document.body.classList.add(styles.macos);
-    } else {
-      document.body.classList.remove(styles.macos);
-    }
-  }, [isMacOS]);
-
   if (!mode) {
     return <Redirect to="/light/" />;
   }

diff --git a/pages/app/styles.scss b/pages/app/styles.scss
@@ -20,15 +20,6 @@ body {
     caret-color: transparent !important;
   }
   // stylelint-enable @cloudscape-design/no-implicit-descendant, selector-max-type
-
-  &.macos {
-    // override scrollbars in macos for stable screenshot tests
-    // stylelint-disable-next-line @cloudscape-design/no-implicit-descendant
-    ::-webkit-scrollbar {
-      inline-size: 0;
-      block-size: 0;
-    }
-  }
 }
 
 /*