From 499f72ad2868137359fe58553e15f3f8b290bb21 Mon Sep 17 00:00:00 2001 From: PureBlack <52806702+Pureblackkk@users.noreply.github.com> Date: Tue, 7 Nov 2023 14:16:28 +0800 Subject: [PATCH] fix(packages/maps): fix l7 draw layer bug on tencent map (#2019) --- packages/maps/src/tmap/map.ts | 70 +++++++++++++++++++++++++++++++++-- 1 file changed, 66 insertions(+), 4 deletions(-) diff --git a/packages/maps/src/tmap/map.ts b/packages/maps/src/tmap/map.ts index 10d43394c6..a6c49fa3a8 100644 --- a/packages/maps/src/tmap/map.ts +++ b/packages/maps/src/tmap/map.ts @@ -32,6 +32,10 @@ const EventMap: { export default class TMapService extends BaseMapService { // @ts-ignore protected viewport: IViewport = null; + protected evtCbProxyMap: Map< + string, + Map<(...args: any) => any, (...args: any) => any> + > = new Map(); public handleCameraChanged = () => { // Trigger map change event @@ -123,6 +127,15 @@ export default class TMapService extends BaseMapService { // Set tencent map canvas element position as absolute // @ts-ignore this.map.canvasContainer.style.position = 'absolute'; + + // Set tencent map control layer dom index + // @ts-ignore + const controlParentContainer = this.map.controlManager.controlContainer?.parentNode; + if (controlParentContainer) { + controlParentContainer.style.zIndex = 2; + } + + this.simpleMapCoord.setSize(mapSize); // May be find an integrated event replacing following events @@ -132,6 +145,8 @@ export default class TMapService extends BaseMapService { this.map.on('pitch', this.handleCameraChanged); this.map.on('zoom', this.handleCameraChanged); + this.map.on('mousemove', () => console.log('asdasd')); + // Trigger camera change after init this.handleCameraChanged(); } @@ -148,6 +163,7 @@ export default class TMapService extends BaseMapService { const container = this.map.getContainer(); this.markerContainer = DOM.create('div', 'l7-marker-container', container); this.markerContainer.setAttribute('tabindex', '-1'); + this.markerContainer.style.zIndex = '2'; } public getMarkerContainer(): HTMLElement { @@ -159,19 +175,65 @@ export default class TMapService extends BaseMapService { if (MapServiceEvent.indexOf(type) !== -1) { this.eventEmitter.on(type, handle); } else { + const onProxy = (eventName: string) => { + let cbProxyMap = this.evtCbProxyMap.get(eventName); + + if (!cbProxyMap) { + this.evtCbProxyMap.set(eventName, (cbProxyMap = new Map())); + } + + if (cbProxyMap.get(handle)) { + return; + } + + const handleProxy = (...args: any[]) => { + if ( + args[0] && + typeof args[0] === 'object' && + !args[0].lngLat && + !args[0].lnglat + ) { + args[0].lngLat = args[0].latlng || args[0].latLng; + } + handle(...args); + }; + + cbProxyMap.set(handle, handleProxy); + this.map.on(eventName, handleProxy); + } + if (Array.isArray(EventMap[type])) { EventMap[type].forEach((eventName: string) => { - this.map.on(eventName || type, handle); + onProxy(eventName || type); }); } else { - this.map.on(EventMap[type] || type, handle); + onProxy(EventMap[type] || type); } } } public off(type: string, handle: (...args: any[]) => void): void { - this.map.off(EventMap[type] || type, handle); - this.eventEmitter.off(type, handle); + if (MapServiceEvent.indexOf(type) !== -1) { + this.eventEmitter.off(type, handle); + return; + } + + const offProxy = (eventName: string) => { + const handleProxy = this.evtCbProxyMap.get(type)?.get(handle); + if (!handleProxy) { + return; + } + this.evtCbProxyMap.get(eventName)?.delete(handle); + this.map.off(eventName, handleProxy); + } + + if (Array.isArray(EventMap[type])) { + EventMap[type].forEach((eventName: string) => { + offProxy(eventName || type); + }); + } else { + offProxy(EventMap[type] || type); + } } public once(): void {