Skip to content

fix: replace Add Variable prompt with an inline form - #194

Merged
dazzatronus merged 1 commit into
mainfrom
fix/add-variable-in-app-input
Sep 25, 2026
Merged

dazzatronus merged 1 commit into
mainfrom
fix/add-variable-in-app-input

Conversation

@dazzatronus

@dazzatronus dazzatronus commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Native prompts can throw in supported browser contexts, preventing users from adding merge fields. Collect the name inside the toolbar, preserve active text composition, and return focus to the Add button when the form closes.

Verify: Node 22 npm run verify:ci; Add Variable regression coverage in the extended suite; local browser submission and Escape cancellation restore focus.

@dazzatronus
dazzatronus force-pushed the fix/add-variable-in-app-input branch from 5caa20d to f1cc89f Compare September 24, 2026 06:03
@dazzatronus
dazzatronus force-pushed the fix/add-variable-in-app-input branch from f1cc89f to 39839ce Compare September 25, 2026 02:10
@dazzatronus dazzatronus changed the title fix: replace Add Variable prompt() with in-app form fix: replace Add Variable prompt with an inline form Sep 25, 2026
@dazzatronus
dazzatronus merged commit f6c2974 into main Sep 25, 2026
1 check passed
@dazzatronus
dazzatronus deleted the fix/add-variable-in-app-input branch September 25, 2026 02:14
@dazzatronus dazzatronus self-assigned this Sep 25, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 25, 2026
## [2.22.7](v2.22.6...v2.22.7) (2026-09-25)

### Bug Fixes

* replace Add Variable prompt with an inline form ([#194](#194)) ([f6c2974](f6c2974))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.22.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant