removed coordinate conversion, implemented a "_has_click_handlers" tr…

…ait for the front end to use so unnecessary messages arent sent if there are no python callbacks registered, and flip floppped the guard clause on the _handle_anywidget_dispatch

lonboard/_map.py

@@ -104,9 +104,8 @@ def __init__(
         def _handle_anywidget_dispatch(
             widget: ipywidgets.Widget, msg: Union[str, list, dict], buffers: List[bytes]
         ) -> None:
-            if msg.get("kind") != "on-click":
-                return
-            self._click_handlers(tuple(msg.get("coordinate")))
+            if msg.get("kind") == "on-click":
+                self._click_handlers(tuple(msg.get("coordinate")))
 
         super().__init__(layers=layers, **kwargs)
         self._click_handlers = CallbackDispatcher()
@@ -124,6 +123,7 @@ def on_click(self, callback, remove=False):
             Set to true to remove the callback from the list of callbacks.
         """
         self._click_handlers.register_callback(callback, remove=remove)
+        self._has_click_handlers = len(self._click_handlers.callbacks) > 0
 
     _esm = bundler_output_dir / "index.js"
     _css = bundler_output_dir / "index.css"
@@ -151,6 +151,10 @@ def on_click(self, callback, remove=False):
         once it's been initially rendered.
 
     """
+    _has_click_handlers = t.Bool(default_value=False, allow_none=False).tag(sync=True)
+    """
+    Indicates if a click handler has been registered.
+    """
 
     _height = t.Int(default_value=DEFAULT_HEIGHT, allow_none=True).tag(sync=True)
     """Height of the map in pixels.

src/index.tsx

@@ -179,15 +179,17 @@ function App() {
     // We added this flag to prevent the hover event from firing after a
     // click event.
     if (typeof info.coordinate !== "undefined") {
-      let x = info.coordinate[0];
-      while (x < 180) {
-        x += 360;
+      // let x = info.coordinate[0];
+      // while (x < 180) {
+      //   x += 360;
+      // }
+      // while (x > 180) {
+      //   x -= 360;
+      // }
+      // info.coordinate[0] = x;
+      if (model.get("_has_click_handlers")) {
+        model.send({ kind: "on-click", coordinate: info.coordinate });
       }
-      while (x > 180) {
-        x -= 360;
-      }
-      info.coordinate[0] = x;
-      model.send({ kind: "on-click", coordinate: info.coordinate });
     }
     setJustClicked(true);
     actorRef.send({