implementing suggestions from Patrick

Author: cpparts

js/models/message_handler.js

@@ -141,7 +141,7 @@ export default class MessageHandler {
   }
 
   handleRemoveOverlay = (msg) => {
-    const overlay_names = msg["name"];
+    const overlay_names = msg["overlay_names"];
     for (const overlay_name of overlay_names) {
       console.info(`Sending removeOverlay for ${overlay_name}`);
       this.aladin.removeOverlay(overlay_name);

src/ipyaladin/widget.py

@@ -985,12 +985,14 @@ def remove_overlay(self, overlay_name: Union[Iterable[str], str]) -> None:
             The string or an iterable of strings.
 
         """
-        overlay_name = [overlay_name] if isinstance(overlay_name, str) else overlay_name
+        overlay_names = (
+            [overlay_name] if isinstance(overlay_name, str) else overlay_name
+        )
 
         self.send(
             {
                 "event_name": "remove_overlay",
-                "name": overlay_name,
+                "overlay_names": overlay_names,
             }
         )
 
@@ -1054,7 +1056,8 @@ def set_listener(self, listener_type: str, callback: Callable) -> None:
         Parameters
         ----------
         listener_type : str
-            Can either be 'object_hovered', 'object_clicked', 'click' or 'select'
+            Can either be 'object_hovered', 'object_clicked', 'click', 'select',
+            or 'current_overlays'
         callback : Callable
             A python function to be called when the event corresponding to the
             listener_type is detected
@@ -1073,7 +1076,8 @@ def set_listener(self, listener_type: str, callback: Callable) -> None:
         else:
             raise ValueError(
                 "listener_type must be 'object_hovered', "
-                "'object_clicked', 'click' or 'select'"
+                "'object_clicked', 'click', 'select', "
+                "or 'current_overlays'"
             )
 
     @widget_should_be_loaded

src/tests/test_aladin.py

@@ -265,7 +265,12 @@ def test_remove_overlay(
     mock_send = Mock()
     monkeypatch.setattr(Aladin, "send", mock_send)
     aladin.remove_overlay(overlay_names)
-    name_info = mock_send.call_args[0][0]["name"]
+
+    event_name = mock_send.call_args[0][0]["event_name"]
+    assert isinstance(event_name, str)
+    assert event_name == "remove_overlay"
+
+    name_info = mock_send.call_args[0][0]["overlay_names"]
     assert isinstance(name_info, list)
     assert name_info[0] in overlay_names