From 15ad774d091fc5cf91de36b406d655e883fe8fd4 Mon Sep 17 00:00:00 2001
From: Fabien Rassinier <frassinier@talend.com>
Date: Tue, 17 Sep 2019 16:31:23 +0200
Subject: [PATCH] feat(components/tabbar): be able to disable responsive
 behavior (#2412)

---
 .../Drawer/__snapshots__/Drawer.test.js.snap  |  6 +-
 .../components/src/TabBar/TabBar.component.js | 57 ++++++++++----
 packages/components/src/TabBar/TabBar.scss    | 13 ++--
 .../TabBar/__snapshots__/TabBar.test.js.snap  | 14 +++-
 packages/components/stories/TabBar.js         | 75 ++++++++++++-------
 5 files changed, 112 insertions(+), 53 deletions(-)

diff --git a/packages/components/src/Drawer/__snapshots__/Drawer.test.js.snap b/packages/components/src/Drawer/__snapshots__/Drawer.test.js.snap
index 42409159be1..a98c243fc91 100644
--- a/packages/components/src/Drawer/__snapshots__/Drawer.test.js.snap
+++ b/packages/components/src/Drawer/__snapshots__/Drawer.test.js.snap
@@ -187,11 +187,11 @@ exports[`Drawer should render with tabs 1`] = `
         onKeyDown={[Function]}
       >
         <ul
-          className="tc-tab-bar theme-tc-responsive-tab-bar nav nav-tabs"
+          className="theme-tc-tab-bar tc-tab-bar theme-tc-tab-bar-responsive tc-tab-bar-responsive nav nav-tabs"
           role="tablist"
         >
           <li
-            className=""
+            className="theme-tc-tab-bar-item tc-tab-bar-item"
             role="presentation"
           >
             <button
@@ -209,7 +209,7 @@ exports[`Drawer should render with tabs 1`] = `
             </button>
           </li>
           <li
-            className="active"
+            className="theme-tc-tab-bar-item tc-tab-bar-item active"
             role="presentation"
           >
             <button
diff --git a/packages/components/src/TabBar/TabBar.component.js b/packages/components/src/TabBar/TabBar.component.js
index c65547f9e69..ba075dc2990 100644
--- a/packages/components/src/TabBar/TabBar.component.js
+++ b/packages/components/src/TabBar/TabBar.component.js
@@ -17,6 +17,8 @@ function TabBar(props) {
 
 	const [showDropdown, setShowDropDown] = useState(false);
 
+	const { responsive = true } = props;
+
 	function tryShowDropdown() {
 		const tabContainer = tabBarContainerRef.current;
 		if (tabContainer) {
@@ -43,8 +45,11 @@ function TabBar(props) {
 	}
 
 	useEffect(() => {
-		const resizeListener = window.addEventListener('resize', debounce(showTabBarAndTest, 200));
-		return () => window.removeEventListener('resize', resizeListener);
+		if (responsive) {
+			const resizeListener = window.addEventListener('resize', debounce(showTabBarAndTest, 200));
+			return () => window.removeEventListener('resize', resizeListener);
+		}
+		return undefined;
 	}, []);
 
 	useEffect(() => {
@@ -62,7 +67,9 @@ function TabBar(props) {
 	});
 
 	useEffect(() => {
-		tryShowDropdown();
+		if (responsive) {
+			tryShowDropdown();
+		}
 	}, []);
 
 	const { onSelect } = props; // to avoid react/no-unused-prop-types
@@ -104,19 +111,28 @@ function TabBar(props) {
 		</Tab.Content>
 	);
 
-	if (showDropdown) {
+	if (responsive && showDropdown) {
 		return (
 			<React.Fragment>
 				<form>
-					<div className={theme['tc-responsive-tab-bar-select-container']}>
-						<select onChange={event => handleSelect(event.target.value, event)} value={selectedKey}>
-							{items.map(item => (
-								<option value={item.key} key={item.key}>
-									{item.label}
-								</option>
-							))}
-						</select>
-					</div>
+					<select
+						className={classnames(theme['tc-tab-bar-dropdown'], 'tc-tab-bar-dropdown')}
+						onChange={event => handleSelect(event.target.value, event)}
+						value={selectedKey}
+					>
+						{items.map(item => (
+							<option
+								className={classnames(
+									theme['tc-tab-bar-dropdown-item'],
+									'tc-tab-bar-dropdown-item',
+								)}
+								value={item.key}
+								key={item.key}
+							>
+								{item.label}
+							</option>
+						))}
+					</select>
 				</form>
 				{tabContent}
 			</React.Fragment>
@@ -134,11 +150,21 @@ function TabBar(props) {
 			<div ref={tabBarContainerRef}>
 				<Nav
 					bsStyle="tabs"
-					className={classnames('tc-tab-bar', theme['tc-responsive-tab-bar'])}
+					className={classnames(
+						theme['tc-tab-bar'],
+						'tc-tab-bar',
+						responsive && theme['tc-tab-bar-responsive'],
+						responsive && 'tc-tab-bar-responsive',
+					)}
 					ref={tabBarRef}
 				>
 					{items.map(item => (
-						<NavItem {...item} eventKey={item.key} componentClass="button">
+						<NavItem
+							className={classnames(theme['tc-tab-bar-item'], 'tc-tab-bar-item')}
+							{...item}
+							eventKey={item.key}
+							componentClass="button"
+						>
 							{item.label}
 						</NavItem>
 					))}
@@ -164,6 +190,7 @@ TabBar.propTypes = {
 		}).isRequired,
 	).isRequired,
 	onSelect: PropTypes.func.isRequired,
+	responsive: PropTypes.bool,
 	selectedKey: PropTypes.any,
 };
 
diff --git a/packages/components/src/TabBar/TabBar.scss b/packages/components/src/TabBar/TabBar.scss
index 89b829f083b..0362e81254f 100644
--- a/packages/components/src/TabBar/TabBar.scss
+++ b/packages/components/src/TabBar/TabBar.scss
@@ -10,17 +10,16 @@
 	background-image: url('data:image/svg+xml;utf8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16"><path d="M8 13.6l-8-8L2.6 3 8 8.4 13.4 3 16 5.6z" fill="#{hex-to-url($color)}"/></svg>');
 }
 
-.tc-responsive-tab-bar {
-	overflow: hidden;
+.tc-tab-bar {
 	white-space: nowrap;
+	overflow: hidden;
 
-	li {
+	& &-item {
 		float: none;
 		display: inline-block;
 	}
-}
-.tc-responsive-tab-bar-select-container {
-	select {
+
+	&-dropdown {
 		@include caret-down($st-tropaz);
 		background-position: right center;
 		background-repeat: no-repeat;
@@ -35,7 +34,7 @@
 		appearance: none;
 		padding: 13px 30px 12px 20px;
 
-		option {
+		&-item {
 			color: initial;
 		}
 	}
diff --git a/packages/components/src/TabBar/__snapshots__/TabBar.test.js.snap b/packages/components/src/TabBar/__snapshots__/TabBar.test.js.snap
index 7c8bedf7f8a..477490288f8 100644
--- a/packages/components/src/TabBar/__snapshots__/TabBar.test.js.snap
+++ b/packages/components/src/TabBar/__snapshots__/TabBar.test.js.snap
@@ -12,7 +12,7 @@ exports[`TabBar component should render with selected children from item definit
     <Nav
       bsClass="nav"
       bsStyle="tabs"
-      className="tc-tab-bar theme-tc-responsive-tab-bar"
+      className="theme-tc-tab-bar tc-tab-bar theme-tc-tab-bar-responsive tc-tab-bar-responsive"
       justified={false}
       pullLeft={false}
       pullRight={false}
@@ -20,6 +20,7 @@ exports[`TabBar component should render with selected children from item definit
     >
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.1"
         disabled={false}
@@ -30,6 +31,7 @@ exports[`TabBar component should render with selected children from item definit
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.2"
         disabled={false}
@@ -40,6 +42,7 @@ exports[`TabBar component should render with selected children from item definit
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.3"
         disabled={false}
@@ -50,6 +53,7 @@ exports[`TabBar component should render with selected children from item definit
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.4"
         disabled={false}
@@ -60,6 +64,7 @@ exports[`TabBar component should render with selected children from item definit
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.5"
         disabled={false}
@@ -138,7 +143,7 @@ exports[`TabBar component should render with selected children managed by user 1
     <Nav
       bsClass="nav"
       bsStyle="tabs"
-      className="tc-tab-bar theme-tc-responsive-tab-bar"
+      className="theme-tc-tab-bar tc-tab-bar theme-tc-tab-bar-responsive tc-tab-bar-responsive"
       justified={false}
       pullLeft={false}
       pullRight={false}
@@ -146,6 +151,7 @@ exports[`TabBar component should render with selected children managed by user 1
     >
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.1"
         disabled={false}
@@ -156,6 +162,7 @@ exports[`TabBar component should render with selected children managed by user 1
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.2"
         disabled={false}
@@ -166,6 +173,7 @@ exports[`TabBar component should render with selected children managed by user 1
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.3"
         disabled={false}
@@ -176,6 +184,7 @@ exports[`TabBar component should render with selected children managed by user 1
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.4"
         disabled={false}
@@ -186,6 +195,7 @@ exports[`TabBar component should render with selected children managed by user 1
       </NavItem>
       <NavItem
         active={false}
+        className="theme-tc-tab-bar-item tc-tab-bar-item"
         componentClass="button"
         data-feature="action.5"
         disabled={false}
diff --git a/packages/components/stories/TabBar.js b/packages/components/stories/TabBar.js
index fa41acd642a..08f554caba4 100644
--- a/packages/components/stories/TabBar.js
+++ b/packages/components/stories/TabBar.js
@@ -1,4 +1,4 @@
-import React from 'react';
+import React, { useState } from 'react';
 import { storiesOf } from '@storybook/react';
 import { action } from '@storybook/addon-actions';
 
@@ -59,34 +59,47 @@ function generateChildId(key, kind) {
 	return null;
 }
 
-class InteractiveTabs extends React.Component {
-	constructor(props) {
-		super(props);
-		this.state = { selectedKey: '2' };
-		this.onSelect = this.onSelect.bind(this);
-	}
-	onSelect(event, item) {
-		this.setState({ selectedKey: item.key });
-	}
-	render() {
-		return (
-			<TabBar {...this.props} selectedKey={this.state.selectedKey} onSelect={this.onSelect}>
-				I'm the child of tab {this.state.selectedKey}
-			</TabBar>
-		);
-	}
+const InteractiveTabs = props => {
+	const [selectedKey, setSelectedKey] = useState('2');
+
+	return (
+		<TabBar
+			{...props}
+			selectedKey={selectedKey}
+			onSelect={(event, item) => setSelectedKey(item.key)}
+		>
+			I'm the child of tab {selectedKey}
+		</TabBar>
+	);
+};
+
+function renderContent() {
+	return (
+		<div style={{ padding: '1rem 2rem' }}>
+			<p>I'm the child</p>
+		</div>
+	);
 }
+
 const stories = storiesOf('TabBar', module);
 stories
 	.add('default', () => (
 		<nav>
 			<h3>Default TabBar</h3>
 			<div id="default">
-				<TabBar {...tabProps}>I'm the child</TabBar>
+				<TabBar {...tabProps}>{renderContent()}</TabBar>
 			</div>
 			<h3>Default TabBar with too small container</h3>
-			<div id="default-smaller" style={{ width: '300px' }}>
-				<TabBar {...tabProps}>I'm the child</TabBar>
+			<div id="default-smaller" style={{ width: '30rem', border: '1px solid' }}>
+				<TabBar {...tabProps}>{renderContent()}</TabBar>
+			</div>
+			<h3>
+				Default TabBar with too small container and <code>responsive = false</code>
+			</h3>
+			<div id="default-smaller" style={{ width: '47rem', border: '1px solid' }}>
+				<TabBar {...tabProps} responsive={false}>
+					{renderContent()}
+				</TabBar>
 			</div>
 		</nav>
 	))
@@ -95,8 +108,10 @@ stories
 			<h3>TabBar with custom ids</h3>
 			<p>
 				By default, you pass an id (required for accessibility) to the component and all ids are
-				generated.<br />
-				But you can customize the ids.<br />
+				generated.
+				<br />
+				But you can customize the ids.
+				<br />
 				The generated id will be passed to the panel as aria-describedby.
 				<pre>{`
 function generateChildId(key, kind) {
@@ -125,11 +140,19 @@ function generateChildId(key, kind) {
 			<h3>TabBar with custom content</h3>
 			<div id="customContent">
 				<TabBar {...tabProps} generateChildId={generateChildId} />
-				<div id="my-custom-id-1" style={{ display: 'none' }}>I'm the existing content of tab 1</div>
+				<div id="my-custom-id-1" style={{ display: 'none' }}>
+					I'm the existing content of tab 1
+				</div>
 				<div id="my-custom-id-2">I'm the existing content of tab 2</div>
-				<div id="my-custom-id-3" style={{ display: 'none' }}>I'm the existing content of tab 3</div>
-				<div id="my-custom-id-4" style={{ display: 'none' }}>I'm the existing content of tab 4</div>
-				<div id="my-custom-id-5" style={{ display: 'none' }}>I'm the existing content of tab 5</div>
+				<div id="my-custom-id-3" style={{ display: 'none' }}>
+					I'm the existing content of tab 3
+				</div>
+				<div id="my-custom-id-4" style={{ display: 'none' }}>
+					I'm the existing content of tab 4
+				</div>
+				<div id="my-custom-id-5" style={{ display: 'none' }}>
+					I'm the existing content of tab 5
+				</div>
 			</div>
 		</nav>
 	))