Skip to content

Share table sidebar shell and scroll body - #8249

Merged
BillLeoutsakosvl346 merged 3 commits into
codex/control-fix-workflow-preview-actionsfrom
codex/emcn-next-table-sidebar
Sep 24, 2026
Merged

BillLeoutsakosvl346 merged 3 commits into
codex/control-fix-workflow-preview-actionsfrom
codex/emcn-next-table-sidebar

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Share the identical sliding shell used by column, workflow, and enrichment table sidebars.
  • Share their identical scroll-body styling while leaving each form and footer in place.
  • Preserve the existing dialog labels, open/closed behavior, 400px width, scroll behavior, and shadow.

Base and verification

Base: #8065 (319f8bb2a9). This change and its focused regression test are on that head.

  • Focused table tests, including a new dialog/transition/scroll-state test, and final-head app TypeScript pass; changed-file Biome check passes.
  • 12 before/after pairs cover light/dark, 16px/20px root text, and open/closed/scrolled states. The captures and computed geometry match.
  • The local design diff checker 3.10.7 reports zero findings or coverage failures against the exact PR base. The full base/head scanner comparison is being completed outside the product PR.

Screenshots and review artifacts are held outside the product checkout. No scanner code or rules are changed here.

@vercel

vercel Bot commented Sep 24, 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 24, 2026 7:15pm UTC

Request Review

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile review this PR

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 24, 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.

@greptile-apps

greptile-apps Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no outstanding finding or new actionable issue remains.

Summary

The PR extracts the common sliding shell and scroll-body styling used by the column, workflow, and enrichment table sidebars while retaining their individual forms and footers. A focused DOM test now covers the shared shell’s visibility transition and mounted content.

Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart TD
  C[Column sidebar] --> S[TableSidebarShell]
  W[Workflow sidebar] --> S
  E[Enrichments sidebar] --> S
  C --> B[TableSidebarScrollBody]
  W --> B
  EC[Enrichment configuration] --> B
Loading

Reviews (2) · Last reviewed commit: "test(tables): preserve sidebar scroll st..."

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile review this PR

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 24, 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 6 files

Confidence score: 5/5

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

Re-trigger cubic

@BillLeoutsakosvl346
BillLeoutsakosvl346 merged commit 5ca5a29 into codex/control-fix-workflow-preview-actions Sep 24, 2026
5 checks passed
BillLeoutsakosvl346 added a commit that referenced this pull request Sep 24, 2026
* improvement(ui): share workflow preview actions

* improvement(ui): reuse inline and overlay workflow actions

* refactor(emcn): share Code.Viewer appearances (#8238)

* Add shared Code.Viewer appearances for product surfaces

* Test flat virtualized Code.Viewer terminal chrome

---------

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

* Share table sidebar shell and scroll body (#8249)

* Share table sidebar shell and scroll body

* test(tables): cover shared sidebar dialog state

* test(tables): preserve sidebar scroll state

---------

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

* Keep workflow preview values readable (#8256)

* fix(ui): keep preview values readable and restore version tint

* fix(ui): keep preview file actions inert

---------

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

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Waleed <walif6@gmail.com>
BillLeoutsakosvl346 added a commit that referenced this pull request Sep 24, 2026
* refactor(ui): centralize overlay action buttons in EMCN

* improvement(ui): share workflow preview actions (#8065)

* improvement(ui): share workflow preview actions

* improvement(ui): reuse inline and overlay workflow actions

* refactor(emcn): share Code.Viewer appearances (#8238)

* Add shared Code.Viewer appearances for product surfaces

* Test flat virtualized Code.Viewer terminal chrome

---------

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

* Share table sidebar shell and scroll body (#8249)

* Share table sidebar shell and scroll body

* test(tables): cover shared sidebar dialog state

* test(tables): preserve sidebar scroll state

---------

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

* Keep workflow preview values readable (#8256)

* fix(ui): keep preview values readable and restore version tint

* fix(ui): keep preview file actions inert

---------

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

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Waleed <walif6@gmail.com>

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Waleed <walif6@gmail.com>
BillLeoutsakosvl346 added a commit that referenced this pull request Sep 24, 2026
* improvement(emcn): share ordinary action geometry in Button

* improvement(ui): share additional-field disclosures

* fix(ui): preserve tiny actions and use shared sidebar geometry

* improvement(ui): use shared geometry for standard icon actions (#8153)

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

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

* refactor(ui): centralize overlay action buttons in EMCN

* improvement(ui): share workflow preview actions (#8065)

* improvement(ui): share workflow preview actions

* improvement(ui): reuse inline and overlay workflow actions

* refactor(emcn): share Code.Viewer appearances (#8238)

* Add shared Code.Viewer appearances for product surfaces

* Test flat virtualized Code.Viewer terminal chrome

---------

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

* Share table sidebar shell and scroll body (#8249)

* Share table sidebar shell and scroll body

* test(tables): cover shared sidebar dialog state

* test(tables): preserve sidebar scroll state

---------

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

* Keep workflow preview values readable (#8256)

* fix(ui): keep preview values readable and restore version tint

* fix(ui): keep preview file actions inert

---------

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

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Waleed <walif6@gmail.com>

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Waleed <walif6@gmail.com>

* improvement(ui): reuse shared button and focus treatments (#8187)

* improvement(ui): reuse shared button and focus treatments

* fix(emcn): include outline reset in muted focus ring

* fix(ui): make closed table sidebars inert and strengthen focus ring

* Carry code search overlay into the live EMCN stack (#8275)

* refactor(ui): share code search overlay controls

* refactor(emcn): own compact code search field chrome

* test(ui): cover floating compact code search

* refactor(emcn): compose compact search from shared chip surface

---------

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

* fix(ui): announce code search result changes

* docs(ui): describe shared table sidebar exports

---------

Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Waleed <walif6@gmail.com>
@waleedlatif1
waleedlatif1 deleted the codex/emcn-next-table-sidebar branch September 24, 2026 23:57

This branch was previously deployed

1 inactive deployment
Preview — eed1bbaf Deployed Sep 24, 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