From 4ea2c3f45094c82016c2888aad4094119620712d Mon Sep 17 00:00:00 2001 From: hieplq Date: Fri, 18 May 2018 15:05:28 +0700 Subject: [PATCH] IDEMPIERE-3552: fix context menu don't show when disable combobox --- .../src/org/adempiere/webui/component/Combobox.java | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/Combobox.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/Combobox.java index 63843c5648..a6c2121a6e 100644 --- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/Combobox.java +++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/Combobox.java @@ -45,13 +45,9 @@ public class Combobox extends org.zkoss.zul.Combobox implements IdSpace } private void override() { - this.setWidgetOverride("slideDown_", "function(pp) {" - + " if (this.isReadonly()) return; " - + " this.$slideDown_(pp); " - + "}"); - this.setWidgetOverride("doKeyDown_", "function(evt) {" - + " if (this.isReadonly()) return; " - + " this.$doKeyDown_(evt); " + // idempiere always want to show context ever on disable control + this.setWidgetOverride("shallIgnoreClick_", "function(evt) {" + + " return false; " + "}"); } @@ -62,7 +58,7 @@ public class Combobox extends org.zkoss.zul.Combobox implements IdSpace public void setEnabled(boolean enabled) { - this.setReadonly(!enabled); + this.setDisabled(!enabled); if (!enabled) { //ensure list is close and not on focus if (this.getPage() != null)