Move Frame classnames inline

Shopify · Apr 10, 2024 · 39acc63 · 39acc63
1 parent ed8142f
commit 39acc63
Showing 1 changed file with 11 additions and 13 deletions.
diff --git a/polaris-react/src/components/Frame/Frame.tsx b/polaris-react/src/components/Frame/Frame.tsx
@@ -240,18 +240,6 @@ class FrameInner extends PureComponent<CombinedProps, State> {
         }
       : {};
 
-    const getFrameClassName = (features: FeaturesConfig) =>
-      classNames(
-        styles.Frame,
-        features?.dynamicTopBarAndReframe && styles['Frame-TopBarAndReframe'],
-        navigation && styles.hasNav,
-        topBar && styles.hasTopBar,
-        sidebar && styles.hasSidebar,
-        sidebar &&
-          features?.dynamicTopBarAndReframe &&
-          styles['hasSidebar-TopBarAndReframe'],
-      );
-
     const contextualSaveBarMarkup = (
       <UseFeatures>
         {({dynamicTopBarAndReframe}) =>
@@ -299,7 +287,17 @@ class FrameInner extends PureComponent<CombinedProps, State> {
         <UseFeatures>
           {(features) => (
             <div
-              className={getFrameClassName(features)}
+              className={classNames(
+                styles.Frame,
+                features?.dynamicTopBarAndReframe &&
+                  styles['Frame-TopBarAndReframe'],
+                navigation && styles.hasNav,
+                topBar && styles.hasTopBar,
+                sidebar && styles.hasSidebar,
+                sidebar &&
+                  features?.dynamicTopBarAndReframe &&
+                  styles['hasSidebar-TopBarAndReframe'],
+              )}
               {...layer.props}
               {...navigationAttributes}
             >