Skip to content

refactor(ui): centralize overlay action buttons in EMCN - #7990

Open
BillLeoutsakosvl346 wants to merge 3 commits into
stagingfrom
codex/control-fix-overlay-action-buttons
Open

BillLeoutsakosvl346 wants to merge 3 commits into
stagingfrom
codex/control-fix-overlay-action-buttons

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Nine product Copy, Search and Expand buttons repeat transparent overlay styling in log details, workflow previews and the deployment preview. Add EMCN OverlayActionButton, built on the existing Button, and migrate all nine locations across four consumer files.

The component owns its 20px/28px geometry, border, background blur and one adaptive hover treatment: surface-3 in light mode and surface-5 in dark mode. The four workflow-preview actions previously used surface-4 on hover; their hover fill changes from #f5f5f5 to #f7f7f7 in light mode and #292929 to #363636 in dark mode. Resting appearance, sizes, corners and behavior stay unchanged. The canonical --border token replaces its equivalent --border-1 alias.

Product code keeps positioning, icons, copy feedback, tooltips, labels and handlers. The specialized component excludes inherited styling props that cannot meaningfully affect its geometry. Existing Button, landing pages and global CSS are unchanged. Audit tooling and screenshots remain outside this PR.

Validation

  • 346 EMCN tests pass across 33 suites, including both preserved adaptive sizes, native ref/prop/event forwarding, tooltip composition and disabled clicks.
  • All 26 workspace type checks and lint checks, API validation, import-specifier checks and diff checks pass.
  • Light/dark browser checks of the real components with compiled application CSS confirm the intended hover colors and unchanged 20px geometry and 5px corners. Matching screenshots and 3× crops are saved externally; this is isolated component coverage, not full product navigation.
  • Product searches find no remaining old overlay recipes or retired overlay surface props. The initial category migration removed all 52 targeted findings; unrelated audit findings remain outside this change.

@vercel

vercel Bot commented Sep 18, 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 19, 2026 4:38pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the shared component preserves the migrated controls’ contracts and the previously missing focused coverage is now present.

Summary

This PR introduces a shared EMCN OverlayActionButton and migrates nine copy, search, and expand controls to it.

  • Centralizes overlay geometry, border, blur, and adaptive hover styling.
  • Preserves consumer-owned positioning, icons, labels, handlers, and tooltip composition.
  • Adds focused coverage for both supported sizes, markup compatibility, ref and native-prop forwarding, keyboard events, and disabled clicks.
  • The previously reported missing-test finding is fully addressed by the current test suite.

Reviews (3) · Last reviewed commit: "refactor(emcn): unify overlay action hov..."

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

@greptileai please re-review PR #7990 at commit 7a463bc. Added four focused OverlayActionButton test cases following the existing action-button conventions: the three migrated appearances (both sizes and hover treatments, with Button class precedence), plus native ref/prop/event forwarding through Tooltip and disabled-click suppression. All 347 EMCN tests, type checks, repository lint and API validation pass. Runtime source is unchanged.

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptileai @cubic-dev-ai Please review the current head. The requested overlay-button test coverage is present and all three focused tests pass.

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

@greptileai @cubic-dev-ai Please review the current head. The requested overlay-button test coverage is present and all three focused tests pass.

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

Confidence score: 5/5

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

Re-trigger cubic

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