chore: minor code cleanup, add comments

geoblocks · Mar 13, 2024 · 3901b9f · 3901b9f
1 parent c29b179
commit 3901b9f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/src/interaction/TrackInteractionModify.ts b/src/interaction/TrackInteractionModify.ts
@@ -64,12 +64,9 @@ export default class Modify extends PointerInteraction {
     type: 'segment'
   });
   private overlay_: VectorLayer<VectorSource<Feature>>;
-  private lastPixel_ = [0, 0];
+  private lastPixel_: Pixel = [0, 0];
   private trackData_: Options['trackData'];
-  /**
-   * @type {Feature<Point>}
-   */
-  private pointAtCursorFeature_ = new Feature({
+  private pointAtCursorFeature_ = new Feature<Point>({
     geometry: new Point([0, 0]),
     type: 'sketch',
     subtype: '',
@@ -126,10 +123,8 @@ export default class Modify extends PointerInteraction {
 
   /**
    * Get the first feature at pixel, favor points over lines
-   * @param {import("ol/pixel").Pixel} pixel
-   * @return {Feature<LineString|Point>|undefined}
    */
-  getFeatureAtPixel(pixel: Pixel): Feature<LineString|Point|undefined> {
+  getFeatureAtPixel(pixel: Pixel): Feature<LineString|Point> {
     const features = this.getMap().getFeaturesAtPixel(pixel, {
       layerFilter: (l) => l.getSource() === this.source_,
       hitTolerance: this.hitTolerance_

diff --git a/src/interaction/TrackManager.ts b/src/interaction/TrackManager.ts
@@ -122,7 +122,7 @@ export default class TrackManager<POIMeta> {
     // }));
 
 
-
+    // Add a control point at the end of the track
     // @ts-ignore too complicate to declare proper events
     this.interaction_.on('drawend',
     async (event: DrawEvent) => {
@@ -168,6 +168,7 @@ export default class TrackManager<POIMeta> {
       }
     });
 
+    // Move an existing poi, control point or segment
     this.interaction_.on(
       // @ts-ignore too complicate to declare proper events
       'modifyend',
@@ -209,6 +210,7 @@ export default class TrackManager<POIMeta> {
         }
     });
 
+    // Delete a control point or a POI
     this.interaction_.on(
       // @ts-ignore too complicate to declare proper events
       'select',