dongjoon-hyun opened a new pull request, #58172:
URL: https://github.com/apache/spark/pull/58172
### What changes were proposed in this pull request?
This PR adds a reusable confirmation prompt for Web UI links:
- `webui.css`: a new `a.confirm-link` style, following the existing
`a.kill-link` style.
- `webui.js`: a CSP-compliant delegated click handler for
`a.confirm-link[data-confirm-message]`
that shows `window.confirm()` with the message and cancels the navigation
when declined.
### Why are the changes needed?
The only existing confirmation pattern, `kill-link`, is tied to the
kill-request form submission.
`confirm-link` guards a plain link navigation instead, so any page can add a
prompt without a
page-specific inline script, which the Web UI no longer allows under CSP.
### Does this PR introduce _any_ user-facing change?
No. No page renders `confirm-link` yet, so the rendered UI is unchanged.
### How was this patch tested?
Pass the CIs, including `dev/lint-js`.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Opus 5
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
