fix: non Event objects do not need processing before serialization

If the event object was a string, we would call pickSerializable
on it which would convert it to an object, which results in a dict
on the Python side.

Author: maartenbreddels

js/src/VueRenderer.js

@@ -75,10 +75,10 @@ function pickSerializable(object, depth=0, max_depth=2) {
 }
 
 export function eventToObject(event) {
-    if (event == null) {
-        return event;
+    if (event instanceof Event) {
+        return pickSerializable(event);
     }
-    return pickSerializable(event);
+    return event;
 }
 
 export function vueRender(createElement, model, parentView, slotScopes) {

tests/ui/test_template.py

@@ -8,7 +8,7 @@
 import playwright.sync_api
 
 from IPython.display import display
-from traitlets import default, Int
+from traitlets import default, Int, Callable, Unicode
 
 
 class MyTemplate(vue.VueTemplate):
@@ -37,3 +37,36 @@ def kernel_code():
     widget.wait_for()
     widget.click()
     page_session.locator("text=Clicked 1").wait_for()
+
+
+class MyEventTemplate(vue.VueTemplate):
+    on_custom = Callable()
+    text = Unicode("Click Me").tag(sync=True)
+
+    @default("template")
+    def _default_vue_template(self):
+        return """
+        <template>
+            <div @click="custom_event('not-an-event-object')">{{text}}</div>
+        </template>
+        """
+
+    def vue_custom_event(self, data):
+        self.on_custom(data)
+
+
+def test_template_custom_event(solara_test, page_session: playwright.sync_api.Page):
+    last_event_data = None
+
+    def on_custom(data):
+        nonlocal last_event_data
+        last_event_data = data
+        div.text = "Clicked"
+
+    div = MyEventTemplate(on_custom=on_custom)
+
+    display(div)
+
+    page_session.locator("text=Click Me").click()
+    page_session.locator("text=Clicked").wait_for()
+    assert last_event_data == "not-an-event-object"