remove aria-busy from root node when target _none (#140)

iamreinder · web-flow · commit a32614c9be4b · 2025-07-02T11:39:43.000-05:00
diff --git a/src/index.js b/src/index.js
@@ -402,6 +402,7 @@ async function send(control, action = '', method = 'GET', body = null, enctype =
   let renders = PendingTargets.get(response).map(async target => {
 
     if (!target.isConnected || target._ajax_id === '_none') {
+      PendingTargets.delete(target)
       return
     }
 
diff --git a/tests/load.cy.js b/tests/load.cy.js
@@ -107,3 +107,19 @@ test('aria-busy is removed from targets that are not replaced',
     })
   }
 )
+
+test('aria-busy is removed from root node when target is _none',
+  html`<html><a href="/tests" x-target="_none">Link</a></html>`,
+  ({ intercept, get, wait }) => {
+    intercept('GET', '/tests', {
+      delay: 1000,
+      statusCode: 200,
+      body: '',
+    }).as('response')
+    get('a').click()
+    get('html').should('have.attr', 'aria-busy')
+    wait('@response').then(() => {
+      get('html').should('not.have.attr', 'aria-busy')
+    })
+  }
+)

Original file line number	Diff line number	Diff line change
`@@ -402,6 +402,7 @@ async function send(control, action = '', method = 'GET', body = null, enctype =`
`402`	`402`	`let renders = PendingTargets.get(response).map(async target => {`
`403`	`403`
`404`	`404`	`if (!target.isConnected \|\| target._ajax_id === '_none') {`
	`405`	`+ PendingTargets.delete(target)`
`405`	`406`	`return`
`406`	`407`	`}`
`407`	`408`