Skip to content

[Content Security] Add a set-field controller for the shared tag modals - #15100

Open
manu-gurudath wants to merge 2 commits into
mainfrom
csp-handler-set-field
Open

manu-gurudath wants to merge 2 commits into
mainfrom
csp-handler-set-field

Conversation

@manu-gurudath

@manu-gurudath manu-gurudath commented Sep 19, 2026

Copy link
Copy Markdown
Member

The two "Create tag" buttons set the shared modal's hidden input through a set-field Stimulus controller instead of an onclick.

馃 Generated with Claude Code

https://claude.ai/code/session_01A8Q14mneHoqkzhDrgWFBFN

export-1789884904543.mp4

Both "Create tag" buttons stamped the record's id into the shared modal's
hidden input from an `onclick`, so the modal knew what it had been opened for.
That becomes `set-field#set`.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A8Q14mneHoqkzhDrgWFBFN
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants