Merge pull request #195 from deyihu/194

drawtool support ground point when clampToModel
dvt3d · Jul 12, 2024 · f7a61aa · f7a61aa
2 parents f2d7757 + f5887bf
commit f7a61aa
Showing 1 changed file with 16 additions and 9 deletions.
diff --git a/src/modules/tools/DrawTool.js b/src/modules/tools/DrawTool.js
@@ -29,15 +29,26 @@ class DrawTool {
     this._tooltipMess = tooltipMess
   }
 
+  _getEventPosition(e) {
+    const { overlay, layer, position, surfacePosition } = e;
+    if (!this._options.clampToModel) {
+      return surfacePosition;
+    }
+    if (!overlay && !layer) {
+      return surfacePosition;
+    }
+    return position;
+
+  }
+
   /**
    *
    * @param e
    * @returns {boolean}
    * @private
    */
   _onClick(e) {
-    let position =
-      this._options.clampToModel && e.position ? e.position : e.surfacePosition
+    let position = this._getEventPosition(e);
     if (!position) {
       return false
     }
@@ -54,8 +65,7 @@ class DrawTool {
    */
   _onMouseMove(e) {
     this._viewer.tooltip.showAt(e.windowPosition, this._tooltipMess)
-    let position =
-      this._options.clampToModel && e.position ? e.position : e.surfacePosition
+    let position = this._getEventPosition(e);
     if (!position) {
       return false
     }
@@ -69,10 +79,7 @@ class DrawTool {
    * @private
    */
   _onRightClick(e) {
-    this._plotEvent.fire(
-      PlotEventType.DRAW_STOP,
-      this._options.clampToModel && e.position ? e.position : e.surfacePosition
-    )
+    this._plotEvent.fire(PlotEventType.DRAW_STOP, this._getEventPosition(e));
   }
 
   /**
@@ -92,7 +99,7 @@ class DrawTool {
         eyeOffset: new Cesium.Cartesian3(0, 0, -100),
         heightReference:
           this._viewer.scene.mode === Cesium.SceneMode.SCENE3D &&
-          !this._options.clampToModel
+            !this._options.clampToModel
             ? Cesium.HeightReference.CLAMP_TO_GROUND
             : Cesium.HeightReference.NONE,
       },