Skip to content

refactor(ui): use EMCN fields in deployment and setup modals - #8009

Open
BillLeoutsakosvl346 wants to merge 2 commits into
codex/emcn-consolidationfrom
codex/control-fix-modal-fields
Open

BillLeoutsakosvl346 wants to merge 2 commits into
codex/emcn-consolidationfrom
codex/control-fix-modal-fields

Conversation

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor

Replace the remaining flagged field wrappers in Chat deployment, MCP deployment, and the Slack setup wizard with ChipModalField. Reuse its labels, spacing, hints, and errors, and allow custom fields to associate their label with an existing control ID.

The fields adopt EMCN’s muted labels, 9px label/control gaps, and aligned side gutters. Existing input surfaces, corners, textarea minimum heights, Slack input height, native required validation, and form submission remain intact. No new component variants are introduced.

Validation:

  • All 8 modal-field audit findings removed; no new styling findings.
  • EMCN: 351 tests passed; relevant consumer suites: 26 tests passed.
  • All workspace type checks and lint checks passed, along with import resolution and mandatory API validation.
  • Browser verification is incomplete: the local application loads the deployment modal, but Chat and MCP data requests leave its fields in loading states. The Slack wizard was not visually verified. No deployment was submitted during this check.

@vercel

vercel Bot commented Sep 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 20, 2026 1:18am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge with no actionable correctness, accessibility, security, or repository-rule issues identified.

Summary

This PR consolidates remaining deployment and Slack setup fields onto ChipModalField while preserving their existing control behavior.

  • Adds optional custom-control label association through htmlFor.
  • Migrates Chat title and welcome-message fields.
  • Migrates MCP tool metadata, parameter descriptions, and server selection.
  • Migrates the Slack wizard’s bot-name field and aligns adjacent layout gutters.
  • Adds coverage for label/error association, native required validation, and form Enter behavior.

Reviews (1) · Last reviewed commit: "refactor(ui): compose deployment and set..."

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

Before / after visual comparison

Chat deployment Title, validation error and Welcome message now use ChipModalField. Labels become muted, field gutters become 8px, and shared label spacing is used; existing input surfaces and the 80px textarea minimum remain.

Left = before; right = after.

Rendered component fixtures with application CSS and local data, rather than authenticated product-page screenshots. These are representative visual examples, not end-to-end behavior coverage.

Fresh isolated render of the chat field compositions; this does not claim live Chat/MCP deployment or Slack setup coverage.

Current PR head checked at posting: 4360284d13.

Before and after: 8009 modal fields

This branch was successfully deployed

1 active deployment
Preview 4360284d Deployed Sep 20, 2026 by vercel[bot]
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.

1 participant