Fix new layer in map (#26)

* Fix the layer position in tree * Do not update a layer not in the tree * Fix the layer position in Map after layerTree update * linter
geojupyter · Jul 2, 2024 · b9d1c78 · b9d1c78
1 parent 40ddf32
commit b9d1c78
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/packages/base/src/mainview/mainview.tsx b/packages/base/src/mainview/mainview.tsx
@@ -293,7 +293,7 @@ export class MainView extends React.Component<IProps, IStates> {
     const currentLayerIds = this._Map.getStyle().layers.map(layer => layer.id);
     let beforeId: string | undefined = undefined;
     if (!(index === undefined) && index < currentLayerIds.length) {
-      beforeId = currentLayerIds[index + 1];
+      beforeId = currentLayerIds[index];
     }
     this._Map.moveLayer(id, beforeId);
   }
@@ -373,7 +373,11 @@ export class MainView extends React.Component<IProps, IStates> {
       if (!layer) {
         this.removeLayer(change.id);
       } else {
-        this.updateLayer(change.id, layer);
+        if (
+          JupyterGISModel.getOrderedLayerIds(this._model).includes(change.id)
+        ) {
+          this.updateLayer(change.id, layer);
+        }
       }
     });
   }

diff --git a/packages/schema/src/model.ts b/packages/schema/src/model.ts
@@ -332,7 +332,7 @@ export class JupyterGISModel implements IJupyterGISModel {
       this._sharedModel.updateLayerTreeItem(mainGroupIndex, mainGroup);
     } else {
       this.sharedModel.addLayerTreeItem(
-        index ?? this.getLayerTree.length,
+        index ?? this.getLayerTree().length,
         item
       );
     }