bterkuile/fizzy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix beforeunload event listeners

Browse Source
This commit is contained in:
Jay Ohms
2026-02-26 07:41:51 -05:00
parent 2d41463074
commit 84c650ab25
2 changed files with 16 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/javascript/controllers/bridge/buttons_controller.js
+8 -8
View File
@@ -6,13 +6,17 @@ export default class extends BridgeComponent {
static targets = [ "button" ]
connect() {
window.addEventListener("beforeunload", this.handleBeforeUnload.bind(this))
window.addEventListener("turbo:before-visit", this.handleBeforeVisit.bind(this))
if (!this.beforeUnloadHandler) {
this.beforeUnloadHandler = this.handleBeforeUnload.bind(this)
}
window.addEventListener("beforeunload", this.beforeUnloadHandler)
}
disconnect() {
window.removeEventListener("beforeunload", this.handleBeforeUnload.bind(this))
window.removeEventListener("turbo:before-visit", this.handleBeforeVisit.bind(this))
if (this.beforeUnloadHandler) {
window.removeEventListener("beforeunload", this.beforeUnloadHandler)
}
}
buttonTargetConnected() {
@@ -45,10 +49,6 @@ export default class extends BridgeComponent {
this.notifyBridgeOfDisconnect()
}
handleBeforeVisit() {
this.notifyBridgeOfDisconnect()
}
#clickButton(message) {
const selectedIndex = message.data.selectedIndex
this.#enabledButtonTargets[selectedIndex].click()
app/javascript/controllers/bridge/form_controller.js
+8 -8
View File
@@ -7,13 +7,17 @@ export default class extends BridgeComponent {
static values = { submitTitle: String }
connect() {
window.addEventListener("beforeunload", this.handleBeforeUnload.bind(this))
window.addEventListener("turbo:before-visit", this.handleBeforeVisit.bind(this))
if (!this.beforeUnloadHandler) {
this.beforeUnloadHandler = this.handleBeforeUnload.bind(this)
}
window.addEventListener("beforeunload", this.beforeUnloadHandler)
}
disconnect() {
window.removeEventListener("beforeunload", this.handleBeforeUnload.bind(this))
window.removeEventListener("turbo:before-visit", this.handleBeforeVisit.bind(this))
if (this.beforeUnloadHandler) {
window.removeEventListener("beforeunload", this.beforeUnloadHandler)
}
}
submitTargetConnected() {
@@ -63,10 +67,6 @@ export default class extends BridgeComponent {
this.notifyBridgeOfDisconnect()
}
handleBeforeVisit() {
this.notifyBridgeOfDisconnect()
}
#observeSubmitTarget() {
this.submitObserver = new MutationObserver(() => {
this.send(this.submitTarget.disabled ? "submitDisabled" : "submitEnabled")