Skip to content

refactor(resources): share filter panel structure - #8260

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/emcn-next-resource-filters
Sep 25, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
codex/emcn-next-resource-filters

Conversation

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor

Summary

Share the exact resource-filter panel and field-section structure across Files, Tables, and document chunks. ResourceOptions continues to own the trigger and popover. The document status label retains its distinct typography; each page keeps its filter state, combobox props, and clear action.

Validation

  • Comparison base: staging at d9db35e0a9376de1562abb1979826b522611a51a.
  • Focused resource-options.test.tsx: 4 passed, including popover open, section focus, and clear action.
  • Biome check and git diff --check: passed.
  • Design diff checker 3.10.7 against the pinned staging base: 0 findings, 0 review items, and 0 coverage failures.
  • App tsc --noEmit: passed with this worktree's own @sim/* workspace package links.
  • Source-derived isolated fixtures with actual EMCN controls: all three panels have matching computed geometry and pixel-identical before/after captures in light and dark, at 16px and 20px root size, in normal and keyboard-focused states. The first combobox receives keyboard focus in the focused captures. Evidence is stored outside the product checkout at ~/.local/state/sim2/design-reviews/resource-filters-20260924/.
  • The aggregate full scan runs on the separate local integration branch.

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@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:40pm UTC

Request Review

@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 actionable regression was identified.

Summary

Extracts the existing filter-panel and section markup into shared components for Files, Tables, and document chunks. The document status label retains its distinct styling; filter state and actions remain page-owned.

Reviews (1) · Last reviewed commit: "refactor(resources): share filter panel ..."

@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

@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

Visual review: resource filter panels

What changed: Files, Tables, and document-chunk filters now share the existing 240px padded panel and stacked label/control structure. Each page keeps its filter choices, state, clear action, and the document status label’s distinct typography.

// Before — Files filter, opening of the File Type section
<div className='flex w-[240px] flex-col gap-3 p-3'>
  <div className='flex flex-col gap-1.5'>
    <span className={FILTER_SECTION_LABEL_CLASS}>File Type</span>

// After — the same section and unchanged ChipCombobox
<ResourceFilterPanel>
  <ResourceFilterSection label='File Type'>

What to inspect: Popover width, section spacing, label type, control placement, and keyboard focus. These are labeled isolated fixtures using source-exact panel/section classes, representative actual EMCN controls, and compiled product CSS; they are not authenticated Files or Tables route captures. The fixture compares staging d9db35e0a9 with current PR head 498718c975e2. All light/dark × 16px/20px × normal/focused screenshot pairs are byte-identical, and the focus control remains keyboard reachable. Four representative pairs:

Theme / root text / state Before After
Light / 16px / normal Before resource filter, light 16px normal After resource filter, light 16px normal
Dark / 16px / focus Before resource filter, dark 16px focus After resource filter, dark 16px focus
Light / 20px / focus Before resource filter, light 20px focus After resource filter, light 20px focus
Dark / 20px / normal Before resource filter, dark 20px normal After resource filter, dark 20px normal

@waleedlatif1
waleedlatif1 merged commit c6b49d0 into staging Sep 25, 2026
34 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/emcn-next-resource-filters branch September 25, 2026 00:10

This branch was previously deployed

1 inactive deployment
Preview — 498718c9 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.

2 participants