fix(cleanup): code cleanup

ensure the example in documentation doesnt break container barrier
Tripwire · Mar 23, 2017 · 548f281 · 548f281
1 parent 938c67a
commit 548f281
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 22,450 deletions.
diff --git a/src/assets/icon_mainav_dash.svg b/src/assets/icon_mainav_dash.svg
diff --git a/src/assets/icon_mainav_dash_hover.svg b/src/assets/icon_mainav_dash_hover.svg
diff --git a/src/assets/icon_mainav_ops.svg b/src/assets/icon_mainav_ops.svg
diff --git a/src/assets/icon_mainav_ops_hover.svg b/src/assets/icon_mainav_ops_hover.svg
diff --git a/src/components/LeftNav/LeftNav.examples.md b/src/components/LeftNav/LeftNav.examples.md
@@ -3,7 +3,7 @@ Simple Card Example
         const iconDashboard = require('../../assets/icon_mainav_dash_selected.svg');
         const iconOperations = require('../../assets/icon_mainav_ops_selected.svg');
       <div style={{height: '200px'}}>
-      <LeftNav>                
+      <LeftNav style={{maxHeight: '200px', minHeight: '200px'}}>                
             <LeftNav.MenuItem active image={iconDashboard} />      
             <LeftNav.MenuItem image={iconOperations} />
       </LeftNav>
@@ -15,6 +15,8 @@ Use the MenuItem subcomponent to place content.  Use the active prop to set the
 item to active styling.  You can also use an onClick handler to deal with
 navigating the user to the appropriate location.
 
+Manually applied style on this component is only to prevent overrun inside of Styleguide.
+
 #### Supported Fragments ####
 
 ```javascript

diff --git a/src/components/LeftNav/LeftNav.jsx b/src/components/LeftNav/LeftNav.jsx
@@ -6,13 +6,9 @@ class LeftNav extends React.Component {
 
   render () {
     return (
-      <div className='nav__left-container'>
-
-        <nav className='nav__left '>
-          {this.props.children}
-        </nav>
-
-      </div>
+      <nav className='nav__left ' {...this.props}>
+        {this.props.children}
+      </nav>
     )
   }
 }

diff --git a/src/styles/components/left-nav.css b/src/styles/components/left-nav.css
@@ -1,6 +1,4 @@
 @import '../variables.css';
-.nav__left-container {
-}
 .nav__left {
     background: color(var(--cloudburst));
     color: white;