sam/goalsbreakdown
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove duplicate submit handler that caused double task creation

Browse Source 
The Save button already has type=submit which naturally triggers
form submission. The additional click handler dispatching submit
event caused handleTaskSubmit to run twice.
This commit is contained in:
Yuyao Huang 2026-05-09 13:03:03 +08:00
parent 8ede63adcd
commit 6a2e0537ea
2 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
static/js/tasks.js
View File

@ -350,9 +350,6 @@ document.addEventListener("DOMContentLoaded", () => {
document.getElementById("task-modal-close").addEventListener("click", closeTaskModal); document.getElementById("task-modal-close").addEventListener("click", closeTaskModal);
document.getElementById("task-modal-cancel").addEventListener("click", closeTaskModal); document.getElementById("task-modal-cancel").addEventListener("click", closeTaskModal);
document.getElementById("task-form").addEventListener("submit", handleTaskSubmit); document.getElementById("task-form").addEventListener("submit", handleTaskSubmit);
document.getElementById("task-save-btn").addEventListener("click", (e) => {
document.getElementById("task-form").dispatchEvent(new Event("submit", { cancelable: true }));
});
document.getElementById("side-panel-close").addEventListener("click", closeSidePanel); document.getElementById("side-panel-close").addEventListener("click", closeSidePanel);
document.getElementById("save-task-btn").addEventListener("click", saveTask); document.getElementById("save-task-btn").addEventListener("click", saveTask);

2
templates/tasks.html
View File

@ -75,7 +75,7 @@
</div> </div>
<div id="task-error" class="error-message"></div> <div id="task-error" class="error-message"></div>
<div class="modal-actions"> <div class="modal-actions">
<button type="submit" class="btn-primary" id="task-save-btn">Save</button> <button type="submit" class="btn-primary">Save</button>
<button type="button" class="btn-secondary" id="task-modal-cancel">Cancel</button> <button type="button" class="btn-secondary" id="task-modal-cancel">Cancel</button>
</div> </div>
</form> </form>