Merge pull request #6 from salesforce-ux/revert-5-picklistModalIssue

Revert "Fix Picklist in Modal + EventUtil for Testing"

Author: ivansfdc

components/SLDSPicklistBase/index.jsx

@@ -261,12 +261,7 @@ module.exports = React.createClass( {
           this.setState({isOpen:false})
         }
       }
-      /* resolve a double click issue with the modal */
-    } else if (this.props.modal && this.state.isFocused && prevState.isFocused) {
-          if (!this.state.isOpen) {
-              this.setState({isOpen : true});
-          }
-      }
+    }
 
 
     if(this.props.value !== prevProps.value){

components/SLDSPicklistBase/list-item.jsx

@@ -90,7 +90,7 @@ module.exports = React.createClass({
           this.props.onMoveFocus(-1);
         }
       }
-      else if(event.keyCode === KEYS.ENTER ||
+      else if(event.keyCode === KEYS.ENTER || 
           event.keyCode === KEYS.SPACE ){
         EventUtil.trapEvent(event);
         if(this.props.onSelect){
@@ -116,9 +116,7 @@ module.exports = React.createClass({
   },
 
   handleBlur(e) {
-    if (this.props.onBlur &&
-        this.refs.link.getDOMNode() !== e.target) {
-
+    if(this.props.onBlur){
       this.props.onBlur(this.props.index, e.relatedTarget);
     }
   },
@@ -132,15 +130,15 @@ module.exports = React.createClass({
 
   render () {
     return (
-      <li
+      <li 
 
         className={"slds-dropdown__item slds-has-icon slds-has-icon--left slds-theme--"+this.props.theme}
         onMouseDown={this.handleMouseDown}
         tabIndex={-1}>
           <a id={'menu-0-'+this.props.index}
             href=''
             ref='link'
-            className='slds-truncate'
+            className='slds-truncate' 
             onClick={this.handleClick}
             onMouseDown={this.handleMouseDown}
             onKeyDown={this.handleKeyDown}

components/utils/EventUtil.js

@@ -16,12 +16,8 @@ const EventUtil = {
   trapEvent(event){
     event.preventDefault();
     event.stopPropagation();
-    
-    if(event.nativeEvent && event.nativeEvent.preventDefault){
+    if(event.nativeEvent){
       event.nativeEvent.preventDefault();
-    }
-
-    if (event.nativeEvent && event.nativeEvent.stopPropagation){
       event.nativeEvent.stopPropagation();
     }
   },
@@ -31,12 +27,12 @@ const EventUtil = {
   },
 
   trapImmediate(event){
-    if(event.nativeEvent && event.nativeEvent.stopImmediatePropagation){
+    if(event.nativeEvent){
       event.nativeEvent.stopImmediatePropagation();
     }
     EventUtil.trap(event);
   }
 
 };
 
-module.exports = EventUtil;
+module.exports = EventUtil;