From 412717ce0e01c278671e3d0bf1698c3c58f73fb6 Mon Sep 17 00:00:00 2001
From: Guillermo Rauch <rauchg@gmail.com>
Date: Fri, 8 Jul 2016 17:33:39 -0700
Subject: [PATCH] tabs: transform to `Component` for consistency

---
 app/tabs.js | 71 +++++++++++++++++++++++++++++++----------------------
 1 file changed, 42 insertions(+), 29 deletions(-)

diff --git a/app/tabs.js b/app/tabs.js
index 7ca0508a..39ecaa46 100644
--- a/app/tabs.js
+++ b/app/tabs.js
@@ -1,35 +1,48 @@
 import React from 'react';
 import classes from 'classnames';
 
-export default function ({ data = [], borderColor, active, activeMarkers = [], onChange, onClose }) {
-  return <nav style={{ WebkitAppRegion: 'drag' }}>{
-    data.length
-      ? 1 === data.length
-        ? <div className='single'>{ data[0] }</div>
-        : <ul style={{ borderColor }} className='tabs'>
-            {
-              data.map((tab, i) => {
-                const isActive = i === active;
-                const hasActivity = ~activeMarkers.indexOf(i);
-                return <li
-                  key={`tab-${i}`}
-                  className={classes({ is_active: isActive, has_activity: hasActivity })}>
-                    <span
-                      style={{ borderColor: isActive ? borderColor : null }}
-                      onClick={ onChange ? onClick.bind(null, i, onChange, active) : null }>
-                      { tab }
-                    </span>
-                    <i onClick={ onClose ? onClose.bind(null, i) : null }>
-                      <svg className='icon'>
-                        <use xlinkHref='assets/icons.svg#close'></use>
-                      </svg>
-                    </i>
-                </li>;
-              })
-            }
-          </ul>
-      : null
-  }</nav>;
+export default class Tabs extends React.Component {
+
+  render () {
+    const {
+      data = [],
+      borderColor,
+      active,
+      activeMarkers = [],
+      onChange,
+      onClose
+    } = this.props;
+
+    return <nav style={{ WebkitAppRegion: 'drag' }}>{
+      data.length
+        ? 1 === data.length
+          ? <div className='single'>{ data[0] }</div>
+          : <ul style={{ borderColor }} className='tabs'>
+              {
+                data.map((tab, i) => {
+                  const isActive = i === active;
+                  const hasActivity = ~activeMarkers.indexOf(i);
+                  return <li
+                    key={`tab-${i}`}
+                    className={classes({ is_active: isActive, has_activity: hasActivity })}>
+                      <span
+                        style={{ borderColor: isActive ? borderColor : null }}
+                        onClick={ onChange ? onClick.bind(null, i, onChange, active) : null }>
+                        { tab }
+                      </span>
+                      <i onClick={ onClose ? onClose.bind(null, i) : null }>
+                        <svg className='icon'>
+                          <use xlinkHref='assets/icons.svg#close'></use>
+                        </svg>
+                      </i>
+                  </li>;
+                })
+              }
+            </ul>
+        : null
+    }</nav>;
+  }
+
 }
 
 function onClick (i, onChange, active) {