fix(obsidian): remove async from button handler for jQuery UI compatibility

jQuery UI dialog buttons don't work properly with async function handlers.
The Browse button was showing up blank/broken.

Changed Browse button handler from:
- async function with await
- to regular function with .then()/.catch()

This matches the pattern used for the Search button and should make all
three buttons (Search, Browse, Cancel) work properly.
This commit is contained in:
Claude 2025-11-14 06:13:53 +00:00
parent be62dfad10
commit 4e8b9e9b0d
No known key found for this signature in database

View file

@ -48,19 +48,20 @@ function showSearchMethodDialog(elementId, elementType, coordinates) {
closeDialogs("#obsidianNoteLoading");
});
},
"Browse": async function () {
"Browse": function () {
$(this).dialog("close");
try {
const noteData = await promptCreateNewNote(elementId, elementType, coordinates);
showMarkdownEditor(noteData, elementType, elementId, coordinates);
} catch (error) {
if (error.message !== "Cancelled") {
ERROR && console.error("Failed to load note:", error);
tip("Failed to load Obsidian note: " + error.message, true, "error", 5000);
}
}
promptCreateNewNote(elementId, elementType, coordinates)
.then(noteData => {
showMarkdownEditor(noteData, elementType, elementId, coordinates);
})
.catch(error => {
if (error.message !== "Cancelled") {
ERROR && console.error("Failed to load note:", error);
tip("Failed to load Obsidian note: " + error.message, true, "error", 5000);
}
});
},
Cancel: function () {
"Cancel": function () {
$(this).dialog("close");
}
},