dhtmlxmenu now doesn't autohide after clicking on checkboxes/radiobuttons, click handler can specifiy whether the field should be checked/unchecked

phpgwapi/js/egw_action/egw_action.js

@@ -222,9 +222,7 @@ egwAction.prototype.execute = function(_senders, _target)
 		_target == null;
 	}
 
-	this.onExecute.exec(this, _senders, _target);
-
-	return false;
+	return this.onExecute.exec(this, _senders, _target);
 }
 
 /**

phpgwapi/js/egw_action/egw_action_popup.js

@@ -324,7 +324,7 @@ function egwPopupActionImplementation()
 						item.set_onClick(function(elem) {
 							// Copy the "checked" state
 							elem.data.checked = elem.checked;
-							elem.data.execute(_selected, _target);
+							return elem.data.execute(_selected, _target);
 						});
 					}
 					else

phpgwapi/js/egw_action/egw_menu_dhtmlx.js

@@ -23,9 +23,40 @@ function egwMenuImpl(_structure)
 		if (id)
 		{
 			var elem = self.dhtmlxmenu.getUserData(id, 'egw_menu');
+
 			if (elem && elem.onClick)
 			{
-				elem.onClick(elem);
+				if (elem.checkbox)
+				{
+					self.dhtmlxmenu.setContextMenuHideAllMode(false);
+				}
+
+				var res = elem.onClick(elem);
+
+				if (elem.checkbox)
+				{
+					var checked = null;
+					if (res == "checkbox:disable")
+					{
+						checked = false;
+					}
+					if (res == "checkbox:enable")
+					{
+						checked = true;
+					}
+
+					if (checked != null)
+					{
+						if (elem.groupIndex != 0)
+						{
+							self.dhtmlxmenu.setRadioChecked(id, checked);
+						}
+						else
+						{
+							self.dhtmlxmenu.setCheckboxState(id, checked);
+						}
+					}
+				}
 			}
 		}
 	});

phpgwapi/js/egw_action/test/test_action.html

@@ -349,7 +349,9 @@
 							"checkbox": true,
 							"checked": false,
 							"caption": "Test2",
-							"onExecute": alertClicked
+							"onExecute": function() {
+								return "checkbox:disable";
+							}
 						}
 					]
 				);