@antv/l7-component 2.23.0 → 2.23.1

package/lib/popup/layerPopup.d.ts

@@ -17,7 +17,7 @@ export type LayerPopupConfigItem = {
 export interface ILayerPopupOption extends IPopupOption {
     config?: LayerPopupConfigItem[];
     items?: LayerPopupConfigItem[];
-    trigger: 'hover' | 'click';
+    trigger: 'hover' | 'click' | 'touchend' | 'touchstart';
 }
 type LayerMapInfo = {
     onMouseMove?: (layer: ILayer, e: any) => void;
@@ -45,6 +45,12 @@ export default class LayerPopup extends Popup<ILayerPopupOption> {
         featureId: number;
     };
     protected get layerConfigItems(): LayerPopupConfigItem[];
+    /**
+     * 根据环境获取实际的触发事件
+     * 当 trigger 为 'click' 时，移动端使用 'touchend'，PC 端使用 'click'
+     * @protected
+     */
+    protected getActualTriggerEvent(): "hover" | "click" | "touchstart" | "touchend";
     addTo(scene: L7Container): this;
     remove(): this;
     setOptions(option: Partial<ILayerPopupOption>): this;
@@ -60,7 +66,7 @@ export default class LayerPopup extends Popup<ILayerPopupOption> {
      */
     protected unbindLayerEvent(): void;
     protected onLayerMouseMove(layer: ILayer, e: any): void;
-    protected onLayerMouseOut(layer: ILayer, e: any): void;
+    protected onLayerMouseOut(layer: ILayer): void;
     protected onLayerClick: (layer: ILayer, e: any) => void;
     protected onSceneClick: () => void;
     protected onSourceUpdate(): void;

package/lib/popup/layerPopup.js

@@ -78,6 +78,21 @@ class LayerPopup extends _popup.default {
     } = this.popupOption;
     return (_ref = config !== null && config !== void 0 ? config : items) !== null && _ref !== void 0 ? _ref : [];
   }
+
+  /**
+   * 根据环境获取实际的触发事件
+   * 当 trigger 为 'click' 时，移动端使用 'touchend'，PC 端使用 'click'
+   * @protected
+   */
+  getActualTriggerEvent() {
+    const {
+      trigger
+    } = this.popupOption;
+    if (trigger === 'click') {
+      return (0, _l7Utils.isPC)() ? 'click' : 'touchend';
+    }
+    return trigger;
+  }
   addTo(scene) {
     super.addTo(scene);
     this.bindLayerEvent();
@@ -105,7 +120,7 @@ class LayerPopup extends _popup.default {
     return this;
   }
   getDefault(option) {
-    const isHoverTrigger = option.trigger !== 'click';
+    const isHoverTrigger = option.trigger === 'hover';
     return (0, _objectSpread2.default)((0, _objectSpread2.default)({}, super.getDefault(option)), {}, {
       trigger: 'hover',
       followCursor: isHoverTrigger,
@@ -130,6 +145,7 @@ class LayerPopup extends _popup.default {
       trigger,
       closeOnClick
     } = this.popupOption;
+    const actualTrigger = this.getActualTriggerEvent();
     this.layerConfigItems.forEach(configItem => {
       var _layer$getSource;
       const layer = this.getLayerByConfig(configItem);
@@ -148,10 +164,10 @@ class LayerPopup extends _popup.default {
         var _this$mapsService;
         const onLayerClick = this.onLayerClick.bind(this, layer);
         layerInfo.onClick = onLayerClick;
-        layer === null || layer === void 0 || layer.on('click', onLayerClick);
+        layer === null || layer === void 0 || layer.on(actualTrigger, onLayerClick);
         const mapContainer = (_this$mapsService = this.mapsService) === null || _this$mapsService === void 0 ? void 0 : _this$mapsService.getMapContainer();
         if (mapContainer && closeOnClick) {
-          mapContainer.addEventListener('click', this.onSceneClick);
+          mapContainer.addEventListener(actualTrigger, this.onSceneClick);
         }
       }
       const source = layer === null || layer === void 0 || (_layer$getSource = layer.getSource) === null || _layer$getSource === void 0 ? void 0 : _layer$getSource.call(layer);
@@ -167,6 +183,7 @@ class LayerPopup extends _popup.default {
    * @protected
    */
   unbindLayerEvent() {
+    const actualTrigger = this.getActualTriggerEvent();
     this.layerConfigItems.forEach(configItem => {
       var _this$mapsService2;
       const layer = this.getLayerByConfig(configItem);
@@ -187,7 +204,7 @@ class LayerPopup extends _popup.default {
         layer.off('mouseout', onMouseOut);
       }
       if (onClick) {
-        layer.off('click', onClick);
+        layer.off(actualTrigger, onClick);
       }
       if (onSourceUpdate) {
         var _layer$getSource2;
@@ -195,7 +212,7 @@ class LayerPopup extends _popup.default {
       }
       const mapContainer = (_this$mapsService2 = this.mapsService) === null || _this$mapsService2 === void 0 ? void 0 : _this$mapsService2.getMapContainer();
       if (mapContainer) {
-        mapContainer.removeEventListener('click', this.onSceneClick);
+        mapContainer.removeEventListener(actualTrigger, this.onSceneClick);
       }
     });
   }
@@ -214,9 +231,7 @@ class LayerPopup extends _popup.default {
       this.show();
     }
   }
-
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  onLayerMouseOut(layer, e) {
+  onLayerMouseOut(layer) {
     this.setDisplayFeatureInfo(undefined);
     if (this.isShow) {
       this.hide();

package/lib/popup/popup.js

@@ -442,7 +442,6 @@ class Popup extends _eventemitter.EventEmitter {
   isOpen() {
     return !!this.mapsService;
   }
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   getDefault(option) {
     // tslint:disable-next-line:no-object-literal-type-assertion
     return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antv/l7-component",
-  "version": "2.23.0",
+  "version": "2.23.1",
   "description": "Component for L7",
   "license": "MIT",
   "author": "https://github.com/orgs/antvis/people",
@@ -16,13 +16,13 @@
     "@babel/runtime": "^7.7.7",
     "eventemitter3": "^4.0.0",
     "supercluster": "^7.0.0",
-    "@antv/l7-core": "2.23.0",
-    "@antv/l7-layers": "2.23.0",
-    "@antv/l7-utils": "2.23.0"
+    "@antv/l7-core": "2.23.1",
+    "@antv/l7-layers": "2.23.1",
+    "@antv/l7-utils": "2.23.1"
   },
   "devDependencies": {
     "less": "^4.1.3",
-    "@antv/l7-test-utils": "^2.23.0"
+    "@antv/l7-test-utils": "^2.23.1"
   },
   "publishConfig": {
     "access": "public",