Merge pull request #1156 from Syncano/icon-menu_closeOnItemTouchTap

[Icon Menu] adding closeOnItemTouchTap
mui · Jul 13, 2015 · 872fb69 · 872fb69
2 parents 733cc5a + 47ca11e
commit 872fb69
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 6 deletions.
diff --git a/docs/src/app/components/pages/components/icon-menus.jsx b/docs/src/app/components/pages/components/icon-menus.jsx
@@ -109,6 +109,12 @@ class IconMenus extends React.Component {
             type: 'number',
             header: 'default: 200',
             desc: 'Sets the delay in milliseconds before closing the menu when an item is clicked.'
+          },
+          {
+            name: 'closeOnItemTouchTap',
+            type: 'bool',
+            header: 'optional',
+            desc: 'If false, menu will not be closed after tap (default: true).'
           }
         ]
       },

diff --git a/src/menus/icon-menu.jsx b/src/menus/icon-menu.jsx
@@ -32,6 +32,7 @@ let IconMenu = React.createClass({
     onTouchTap: React.PropTypes.func,
     menuStyle: React.PropTypes.object,
     touchTapCloseDelay: React.PropTypes.number,
+    closeOnItemTouchTap: React.PropTypes.bool,
   },
 
   getDefaultProps() {
@@ -46,6 +47,7 @@ let IconMenu = React.createClass({
       onMouseUp: () => {},
       onTouchTap: () => {},
       touchTapCloseDelay: 200,
+      closeOnItemTouchTap: true,
     };
   },
 
@@ -160,14 +162,17 @@ let IconMenu = React.createClass({
   },
 
   _handleItemTouchTap(e, child) {
-    let isKeyboard = Events.isKeyboard(e);
 
-    this._timeout = setTimeout(() => {
-      this.close(isKeyboard);
-    }, this.props.touchTapCloseDelay);
+    if (this.props.closeOnItemTouchTap) {
+      let isKeyboard = Events.isKeyboard(e);
 
-    if (isKeyboard) {
-      this.refs[this.state.iconButtonRef].setKeyboardFocus();
+      this._timeout = setTimeout(() => {
+        this.close(isKeyboard);
+      }, this.props.touchTapCloseDelay);
+
+      if (isKeyboard) {
+        this.refs[this.state.iconButtonRef].setKeyboardFocus();
+      }
     }
 
     this.props.onItemTouchTap(e, child);