IDEMPIERE-5219 Sales Order Window: initial focus not on header tab's field (#1222)

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/ADTabpanel.java

@@ -1416,7 +1416,7 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
         } else {
         	if (activate) {
         		formContainer.setVisible(activate);
-        		if (!isMobile())
+        		if (!isMobile() && !isDetailPaneMode())
         			focusToFirstEditor();
         	}
         }
@@ -2330,5 +2330,10 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
 	{
 
 	}
+	
+	public AbstractADWindowContent getADWindowContent()
+	{
+		return windowPanel;
+	}
 
 }

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/AbstractADWindowContent.java

@@ -1170,11 +1170,30 @@ public abstract class AbstractADWindowContent extends AbstractUIPart implements
 
     /**
      * restore focus to last known focus editor (if any)
+     * @return true if there's last focus editor
      */
-	public void focusToLastFocusEditor() {
+	public boolean focusToLastFocusEditor() {
+		return focusToLastFocusEditor(false);
+	}
+	
+    /**
+     * restore focus to last known focus editor (if any)
+     * @param defer true to schedule for later/defer execution
+     * @return true if there's last focus editor
+     */
+	public boolean focusToLastFocusEditor(boolean defer) {
 		if (lastFocusEditor != null && lastFocusEditor instanceof HtmlBasedComponent && 
-			lastFocusEditor.getPage() != null && LayoutUtils.isReallyVisible(lastFocusEditor))
+			lastFocusEditor.getPage() != null && LayoutUtils.isReallyVisible(lastFocusEditor)) {
-			((HtmlBasedComponent)lastFocusEditor).focus();
+			if (defer) {
+				final HtmlBasedComponent editor = (HtmlBasedComponent) lastFocusEditor;
+				Executions.schedule(getComponent().getDesktop(), e -> editor.focus(), new Event("onScheduleFocusToLastFocusEditor"));
+			} else {
+				((HtmlBasedComponent)lastFocusEditor).focus();
+			}
+			return true;
+		} else {
+			return false;
+		}
 	}
 
 	/**

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/CompositeADTabbox.java

@@ -388,6 +388,10 @@ public class CompositeADTabbox extends AbstractADTabbox
 				if (tabPanel != headerTab && headerTab.getDetailPane() != null) {
 					if (b != null && b.booleanValue()) {
 						onActivateDetail(tabPanel);
+						if (headerTab instanceof ADTabpanel) {
+							if (!((ADTabpanel) headerTab).getADWindowContent().focusToLastFocusEditor(true))
+								((ADTabpanel) headerTab).getADWindowContent().focusToActivePanel();
+						}
 					}
 				}
 			}

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/ComboEditorBox.java

@@ -189,4 +189,9 @@ public class ComboEditorBox extends Div {
 		}
 			
 	}
+
+	@Override
+	public void focus() {
+		txt.focus();
+	}
 }

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/EditorBox.java

@@ -162,4 +162,9 @@ public class EditorBox extends Div {
 		}
 			
 	}
+
+	@Override
+	public void focus() {
+		txt.focus();
+	}
 }

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/component/Urlbox.java

@@ -170,4 +170,9 @@ public class Urlbox extends Div
 		}
 			
 	}
+
+	@Override
+	public void focus() {
+		txt.focus();
+	}
 }