fix issue mui#2482 [Appbar] onLeftIconButtonTouchTap event doesn't fire

pratimary · Sep 22, 2016 · e1ae269 · e1ae269
1 parent 367d000
commit e1ae269
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/docs/src/app/components/pages/components/AppBar/ExampleIconButton.js b/docs/src/app/components/pages/components/AppBar/ExampleIconButton.js
@@ -8,6 +8,10 @@ function handleTouchTap() {
   alert('onTouchTap triggered on the title component');
 }
 
+function handleTouchTapLeftIconButton() {
+  alert('onClick triggered on the left icon component');
+}
+
 const styles = {
   title: {
     cursor: 'pointer',
@@ -19,6 +23,7 @@ const AppBarExampleIconButton = () => (
     title={<span style={styles.title}>Title</span>}
     onTitleTouchTap={handleTouchTap}
     iconElementLeft={<IconButton><NavigationClose /></IconButton>}
+    onLeftIconButtonTouchTap={handleTouchTapLeftIconButton}
     iconElementRight={<FlatButton label="Save" />}
   />
 );

diff --git a/src/AppBar/AppBar.js b/src/AppBar/AppBar.js
@@ -201,7 +201,7 @@ class AppBar extends Component {
 
     const {prepareStyles} = this.context.muiTheme;
     const styles = getStyles(this.props, this.context);
-
+    const self = this;
     let menuElementLeft;
     let menuElementRight;
 
@@ -230,6 +230,7 @@ class AppBar extends Component {
 
           iconElementLeftNode = React.cloneElement(iconElementLeft, {
             iconStyle: Object.assign({}, iconButtonIconStyle, iconElementLeft.props.iconStyle),
+             onClick: self.handleTouchTapLeftIconButton.bind(self),
           });
         }
 
@@ -279,6 +280,7 @@ class AppBar extends Component {
         case 'FlatButton':
           iconElementRightNode = React.cloneElement(iconElementRight, {
             style: Object.assign({}, styles.flatButton, iconElementRight.props.style),
+             onClick: self.handleTouchTapRightIconButton.bind(self),
           });
           break;