Woops, just remove overflow rule

SparkPost · May 29, 2020 · a52a71b · a52a71b
1 parent c5dbf9a
commit a52a71b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 15 deletions.
diff --git a/packages/matchbox/src/components/Tabs/Tabs.js b/packages/matchbox/src/components/Tabs/Tabs.js
@@ -121,12 +121,7 @@ function Tabs(props) {
   }, [selected, tabs, isOverflowing, disableResponsiveBehavior]);
 
   return (
-    <Container
-      borderBottom="400"
-      disableResponsiveBehavior={disableResponsiveBehavior}
-      {...pick(rest)}
-      ref={wrapperRef}
-    >
+    <Container borderBottom="400" {...pick(rest)} ref={wrapperRef}>
       <Box aria-hidden={isOverflowing} overflow="hidden">
         <OverflowTabContainer
           aria-orientation="horizontal"

diff --git a/packages/matchbox/src/components/Tabs/styles.js b/packages/matchbox/src/components/Tabs/styles.js
@@ -51,8 +51,7 @@ export const overflowTabs = ({ isOverflowing }) => `
   pointer-events: ${isOverflowing ? 'none' : 'auto'};
 `;
 
-export const containerStyles = ({ disableResponsiveBehavior }) => `
+export const containerStyles = () => `
   position: relative;
   height: ${tokens.sizing_750};
-  ${!disableResponsiveBehavior ? `overflow: hidden;` : ''}
 `;
diff --git a/packages/matchbox/src/components/Tabs/tests/Tabs.test.js b/packages/matchbox/src/components/Tabs/tests/Tabs.test.js
@@ -53,11 +53,4 @@ describe('Tabs', () => {
     });
     expect(wrapper.find('a').text()).toEqual('Tab 4');
   });
-
-  it('should be able to disable responsive behavior', () => {
-    const wrapper = subject({
-      disableResponsiveBehavior: true,
-    });
-    expect(wrapper).not.toHaveStyleRule('overflow', 'hidden');
-  });
 });