优化：optimize change for combo-select (#540)

* fix #372

* - remove event lisntener as soon as possible;
- change close debounce timeout to 400ms;
- change popup debounce timeout to 100ms;

* - remove event lisntener as soon as possible;
- change close timeout to 400ms;
- change open timeout to 100ms;

* merge change from master

Author: oicqcx

src/jigsaw/component/combo-select/combo-select.ts

@@ -294,11 +294,9 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
         if (this._removeWindowClickHandler) {
             this._removeWindowClickHandler();
         }
-
         this._removeWindowClickHandler = this._renderer.listen('window', 'click', () => {
-            if (this._removeWindowClickHandler) {
-                this._removeWindowClickHandler();
-            }
+            this._removeWindowClickHandler();
+            this._removeWindowClickHandler = null;
             this.open = false
         });
 
@@ -309,13 +307,23 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
         this._disposePopup = popupInfo.dispose;
 
         if (this._openTrigger === DropDownTrigger.mouseenter && this._popupElement) {
+            if (this._removeMouseOverHandler) {
+                this._removeMouseOverHandler();
+            }
             this._removeMouseOverHandler = this._renderer.listen(this._popupElement, 'mouseenter', () => {
+                this._removeMouseOverHandler();
+                this._removeMouseOverHandler = null;
                 clearTimeout(this._rollOutDenouncesTimer);
             });
         }
         if (this._closeTrigger === DropDownTrigger.mouseleave && this._popupElement) {
+            if (this._removeMouseOutHandler) {
+                this._removeMouseOutHandler();
+            }
             this._removeMouseOutHandler = this._renderer.listen(this._popupElement, 'mouseleave', () => {
-                this._rollOutDenouncesTimer = this.callLater(() => this.open = false, 200);
+                this._removeMouseOutHandler();
+                this._removeMouseOutHandler = null;
+                this._rollOutDenouncesTimer = this.callLater(() => this.open = false, 400);
             });
         }
 
@@ -336,6 +344,10 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
             this._disposePopup();
             this._disposePopup = null;
         }
+        this._clearAllListeners();
+    }
+
+    private _clearAllListeners() {
         if (this._removeWindowClickHandler) {
             this._removeWindowClickHandler();
             this._removeWindowClickHandler = null;
@@ -391,7 +403,7 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
         this._rollInDenouncesTimer = setTimeout(() => {
             this.open = true;
             if (this._editor) this._editor.select();
-        }, 200)
+        }, 100);
     }
 
     /**
@@ -403,7 +415,7 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
         event.stopPropagation();
 
         clearTimeout(this._rollInDenouncesTimer);
-        this._rollOutDenouncesTimer = this.callLater(() => this.open = false, 200);
+        this._rollOutDenouncesTimer = this.callLater(() => this.open = false, 400);
     }
 
     /**
@@ -427,8 +439,8 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
     public ngAfterViewInit() {
         this._tags.changes.subscribe(() => {
             this._autoEditorWidth();
-            // 等待combo高度变化，调整下拉位置
             this.callLater(() => {
+                // 等待combo高度变化，调整下拉位置
                 if (this._popupElement) {
                     this._popupService.setPosition(this._getPopupOption(), this._popupElement);
                 }
@@ -440,10 +452,15 @@ export class JigsawComboSelect extends AbstractJigsawComponent implements Contro
         super.ngOnDestroy();
 
         this.open = false;
+        this._clearAllListeners();
 
         if (this._removeRefreshCallback) {
-            this._removeRefreshCallback()
+            this._removeRefreshCallback();
+            this._removeRefreshCallback = null;
         }
+
+        this._popupElement = null;
+        this._disposePopup = null;
     }
 
     private _propagateChange: any = () => {