Add some more todos

MAAP-Project · Aug 2, 2023 · 7863b60 · 7863b60
1 parent 587e79b
commit 7863b60
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 24 deletions.
diff --git a/stac_ipyleaflet/widgets/draw.py b/stac_ipyleaflet/widgets/draw.py
@@ -4,8 +4,10 @@
 # @NOTE: This should be an extension of the IPYLEAFLET Class. Currently it is just being passed
 # in instead due to import errors
 
-
 # @TODO: Fix linting errors caused by inferred inheritance and just pass in params instead
+
+
+# @TODO: Break out shared logic between widgets into a utilities directory
 class DrawControlWidget:
     def template(self, **kwargs) -> Box(style={"max_height: 200px"}):
         main = self
@@ -35,15 +37,16 @@ def template(self, **kwargs) -> Box(style={"max_height: 200px"}):
         tabs = {}
 
         for i in range(2):
-            tabs[f'child{i}'] = (main.interact_widget.children[0]
-            .children[i]
-            .children[0]
-            .children[0]
-            .children
-        )
-
-        point_tab_children = tabs['child0']
-        area_tab_children = tabs['child1']
+            tabs[f"child{i}"] = (
+                main.interact_widget.children[0]
+                .children[i]
+                .children[0]
+                .children[0]
+                .children
+            )
+
+        point_tab_children = tabs["child0"]
+        area_tab_children = tabs["child1"]
 
         aoi_coords = area_tab_children[1]
         aoi_clear_button = area_tab_children[2]
@@ -59,8 +62,10 @@ def handle_draw(self, action, geo_json, **kwargs):
             main.aoi_coordinates = []
             main.aoi_bbox = ()
 
-            if('Coordinates' in point_tab_children[1].value):
-                area_tab_children[1].value = "<code>Waiting for points of interest...</code>"
+            if "Coordinates" in point_tab_children[1].value:
+                area_tab_children[
+                    1
+                ].value = "<code>Waiting for points of interest...</code>"
 
             if action == "created":
                 if geo_json["geometry"]:

diff --git a/stac_ipyleaflet/widgets/inspect.py b/stac_ipyleaflet/widgets/inspect.py
@@ -16,12 +16,15 @@ class LayerData:
 
 # @NOTE: This should be an extension of the IPYLEAFLET Class. Currently it is just being passed
 # in instead due to import errors
+
+
+# @TODO: Break out shared logic between widgets into a utilities directory
 class InspectControlWidget:
     def template(
         self, applied_layers, interact_widget, make_get_request, titiler_endpoint
     ):
         main = self
-        
+
         # @TODO-CLEANUP: Create only one DrawControl and pass in the attributes instead
         draw_control = DrawControl(
             edit=False,
@@ -39,15 +42,16 @@ def template(
         tabs = {}
 
         for i in range(2):
-            tabs[f'child{i}'] = (main.interact_widget.children[0]
-            .children[i]
-            .children[0]
-            .children[0]
-            .children
-        )
-
-        point_tab_children = tabs['child0']
-        area_tab_children = tabs['child1']
+            tabs[f"child{i}"] = (
+                main.interact_widget.children[0]
+                .children[i]
+                .children[0]
+                .children[0]
+                .children
+            )
+
+        point_tab_children = tabs["child0"]
+        area_tab_children = tabs["child1"]
 
         point_data = point_tab_children[1]
         clear_button = point_tab_children[2]
@@ -114,8 +118,10 @@ def handle_clear(event):
                 return
 
             self.coordinates = []
-            if('Coordinates' in area_tab_children[1].value):
-                area_tab_children[1].value = "<code>Waiting for area of interest...</code>"
+            if "Coordinates" in area_tab_children[1].value:
+                area_tab_children[
+                    1
+                ].value = "<code>Waiting for area of interest...</code>"
 
             if action == "created":
                 if geo_json["geometry"] and geo_json["geometry"]["type"] == "Point":