From 6e516aee6a10e86140802a23fe00271d5ffb440e Mon Sep 17 00:00:00 2001
From: Berkeley Martinez <Berkeley@robotie.com>
Date: Thu, 28 Jul 2016 22:48:45 -0700
Subject: [PATCH] Fix(map): map drawer new tab button

---
 common/app/components/Drawer.jsx     | 20 ++++++++++++++++----
 common/app/components/Map-Drawer.jsx |  3 +++
 2 files changed, 19 insertions(+), 4 deletions(-)

diff --git a/common/app/components/Drawer.jsx b/common/app/components/Drawer.jsx
index a2c2f783565..a9208f8ece1 100644
--- a/common/app/components/Drawer.jsx
+++ b/common/app/components/Drawer.jsx
@@ -6,11 +6,21 @@ export default class Drawer extends React.Component {
   static propTypes = {
     children: PropTypes.node,
     isOpen: PropTypes.bool,
-    closeDrawer: PropTypes.func
+    closeDrawer: PropTypes.func,
+    closeAria: PropTypes.string,
+    newTabLink: PropTypes.string,
+    newTabAria: PropTypes.string
   };
 
   render() {
-    const { isOpen, closeDrawer, children } = this.props;
+    const {
+      isOpen,
+      closeDrawer,
+      closeAria,
+      children,
+      newTabAria,
+      newTabLink
+    } = this.props;
     const drawerClass = classnames({
       drawer: true,
       'is-collapsed': !isOpen
@@ -19,11 +29,13 @@ export default class Drawer extends React.Component {
       <aside className={ drawerClass }>
         <div className='drawer-action-bar'>
           <a
-            aria-label='open map in new tab'
+            aria-label={ newTabAria }
             className='drawer-action-item drawer-action-pop-out'
+            href={ newTabLink }
+            target='_blank'
           />
           <button
-            aria-label='close map aside'
+            aria-label={ closeAria }
             className='drawer-action-item drawer-action-collapse'
             onClick={ closeDrawer }
           />
diff --git a/common/app/components/Map-Drawer.jsx b/common/app/components/Map-Drawer.jsx
index 9e287d6c551..5da3f8c6c62 100644
--- a/common/app/components/Map-Drawer.jsx
+++ b/common/app/components/Map-Drawer.jsx
@@ -16,8 +16,11 @@ export default class MapDrawer extends React.Component {
     const { isOpen, isAlreadyLoaded, toggleMapDrawer } = this.props;
     return (
       <Drawer
+        closeAria='close map aside'
         closeDrawer={ toggleMapDrawer }
         isOpen={ isOpen }
+        newTabAria='open map in new tab'
+        newTabLink='/map'
         >
         <NoSSR>
           <div>