fixed callback bug which caused errors on resetHandlers via an update

Added dropdown menu internal buttons and links - closing behaviour
added an autoclose property which can prevent closing on menu internal clicks

src/DropdownDivConverter/DropdownDivConverter.xml

@@ -21,6 +21,11 @@
             <category>General</category>
             <description>Will the menu be open on load?</description>
         </property>
+        <property key="autoClose" type="boolean" defaultValue="true" required="true">
+            <caption>Menu self-close</caption>
+            <category>General</category>
+            <description>If set to true: clicking on the menu or it's buttons will close the dropdown menu.</description>
+        </property>
         <property key="buttonSize" type="enumeration" defaultValue="default">
             <caption>Button size</caption>
             <category>Appearance</category>

src/DropdownDivConverter/widget/DropdownDivConverter.js

@@ -35,6 +35,7 @@ define([
         buttonTitle: "",
         isDropUp: "",
         startOpen: "",
+        autoClose: "",
         splitButtonActive: "",
         splitButtonClicked:"",
 
@@ -131,7 +132,26 @@ define([
                     }
 
                     this._toggleMenu();
-                }));  
+                }));
+
+                // prevent default closing on dropdownMenu if needed
+                if (!this.autoClose){
+                    this.connect(this.dropdownMenu, "click", lang.hitch(this,function(e) {
+                        event.stop(e);
+                    }));
+                }
+
+                // Mendix buttons and links stop events from bubbling: set actions for internal button clicks to close the menu if needed
+                if (this.autoClose){
+                    var internalButtons = domQuery("button, a",this.dropdownMenu);
+                    dojoArray.forEach(internalButtons, lang.hitch(this,function(node, i){
+                        this.connect(node, "click", lang.hitch(this, function(e) {
+                            if (this._isOpen){
+                                this._toggleMenu();   
+                            }
+                        }));
+                    }));
+                }
 
                 // set the action for the possible split group button
                 if (this.splitButtonActive){
@@ -176,7 +196,9 @@ define([
                 }
 
             }
-            callback();
+            if (callback){
+                callback();
+            }
         },
 
         // toggle dropdown state