fix #719

Author: lovasoa

CHANGELOG.md

@@ -20,6 +20,7 @@
     insert into t values (42); -- this next statement should not be executed
     ```
 - Fix `error returned from database: 1295 (HY000): This command is not supported in the prepared statement protocol yet` when trying to use transactions with MySQL. `START TRANSACTION` now works as expected in MySQL.
+- Fix a bug where a multi-select dropdown would unexpectedly open when the form was reset.
 
 ## 0.31.0 (2024-11-24)
 

sqlpage/tomselect.js

@@ -18,7 +18,17 @@ function sqlpage_select_dropdown() {
       },
     });
     if (is_focused) tom.focus();
-    s.form?.addEventListener("reset", () => setTimeout(tom.sync.bind(tom), 0));
+    s.form?.addEventListener("reset", async () => {
+      // The reset event is fired before the form is reset, so we need to wait for the next event loop
+      await new Promise((resolve) => setTimeout(resolve, 0));
+      // Sync the options with the new reset value
+      tom.sync();
+      // Wait for the options to be updated
+      await new Promise((resolve) => setTimeout(resolve, 0));
+      // "sync" also focuses the input, so we need to blur it to remove the focus
+      tom.blur();
+      tom.close();
+    });
   }
 }