Skip to content

refactor(ui): share field disclosures and icon action geometry - #8038

Open
BillLeoutsakosvl346 wants to merge 4 commits into
codex/control-fix-icon-button-sizingfrom
codex/control-fix-field-disclosures
Open

BillLeoutsakosvl346 wants to merge 4 commits into
codex/control-fix-icon-button-sizingfrom
codex/control-fix-field-disclosures

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Share additional-field disclosure rows and ordinary icon-action geometry through existing EMCN components.

  • FieldDisclosure composes Button and dashed divider lines; editor/table consumers retain their labels, state, permission guards, chevron animation and keyboard focus treatment.
  • Table headers, document/image toolbars, microphone/public-chat actions and playground controls use shared sizes. Default desktop 28px/32px and mobile 40px targets are retained. Former fixed 28px controls scale with enlarged text.
  • Toolbar and microphone controls deliberately adopt existing quiet/active resting colours and hover fills, including a brighter inactive microphone in dark mode. Recording animation and handlers are unchanged. Tiny content-sized chat/editor controls remain unchanged.

Validation: all workspace type/import/API/lint checks, the complete EMCN suite and relevant consumer tests pass. Current-head full CI, including application build and PostgreSQL tests, passes. Browser fixtures use actual components with local state for light/dark, selected/disabled/focus/hover, mobile, enlarged text and image drag resizing. No microphone recording, authenticated workspace or backend messaging run was exercised.

Shared-definition notifications and retained tiny-action overrides remain explicit in the external design audit. Review fixtures, linters and screenshots are outside the product diff.

@vercel

vercel Bot commented Sep 19, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 22, 2026 8:27pm 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 outstanding or newly introduced actionable issues.

Summary

This PR consolidates additional-field disclosure rows and icon-action geometry through shared EMCN components.

  • Adds an accessible, controlled FieldDisclosure component with shared divider, chevron, and focus treatment.
  • Introduces reusable table sidebar headers and icon actions.
  • Migrates editor, toolbar, microphone, chat, image-resize, and playground controls to shared button geometry.
  • Retains consumer state, handlers, permission guards, labels, responsive sizing, and recording behavior.

Reviews (6) · Last reviewed commit: "improvement(ui): use shared geometry for..."

Comment thread packages/emcn/src/components/field-divider/field-divider.tsx

@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

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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

Additional-field disclosure rows share the button/divider implementation. Expanded and collapsed treatments retain their spacing and appearance.

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.

Current PR head checked at posting: 2d56098b9b.

Before and after: comparison

@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the codex/control-fix-icon-button-sizing branch from 0be585b to 13f85d8 Compare September 22, 2026 19:28
@BillLeoutsakosvl346
BillLeoutsakosvl346 changed the base branch from codex/control-fix-icon-button-sizing to codex/overrides-button-sizing September 22, 2026 19:57
@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the codex/control-fix-field-disclosures branch from 2d56098 to f472cbb Compare September 22, 2026 19:57
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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.

All reported issues were addressed across 13 files

Tip: instead of fixing issues one by one fix them all with cubic

Re-trigger cubic

Comment thread apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx Outdated
Comment thread apps/sim/app/(interfaces)/chat/components/message/components/file-download.tsx Outdated
Comment thread apps/sim/app/(interfaces)/chat/components/message/message.tsx Outdated
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile Please re-review the current head; each finding has a response and the applicable fixes are pushed.

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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 9 files

Confidence score: 5/5

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

Re-trigger cubic

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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.

All reported issues were addressed across 16 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

@BillLeoutsakosvl346 BillLeoutsakosvl346 changed the title improvement(ui): share additional-field disclosures refactor(ui): share field disclosures and icon action geometry Sep 22, 2026
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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 16 files

Confidence score: 5/5

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

Re-trigger cubic

Base automatically changed from codex/overrides-button-sizing to codex/control-fix-icon-button-sizing September 23, 2026 02:12

This branch was previously deployed

1 inactive deployment
Preview 7bf8fe33 Deployed Sep 22, 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