Skip to content

Adopt shadcn-svelte controls and isolate demo benchmarks - #313

Merged
jsolly merged 1 commit into
mainfrom
codex/adopt-shadcn-svelte
Oct 3, 2026
Merged

jsolly merged 1 commit into
mainfrom
codex/adopt-shadcn-svelte

Conversation

@jsolly

@jsolly jsolly commented Oct 3, 2026

Copy link
Copy Markdown
Owner

The gallery's shared Sort control now uses generated shadcn-svelte NativeSelect components, preserving every sort mode and saved drag order. The ten framework comparison demos retain their original implementations.

Product and measurement routes build in separate entry graphs, then compose into the same deployed output. A build-time import guard prevents gallery controls from contaminating framework payloads. Benchmark data and methodology reflect the served artifacts; the existing AI scores are preserved.

Validation: full local gate; desktop/mobile browser checks for all nine sort modes, drag/reload persistence, and all ten checkbox demos with clean console; production-static/preview byte parity; counterfactual build without shared UI produces byte-identical measured JavaScript; independent benchmark critic and full ship review passed.

@coderabbitai

coderabbitai Bot commented Oct 3, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4b0cdd04-73d5-460e-b9c7-24c069e86e31
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jsolly
jsolly enabled auto-merge (squash) October 3, 2026 23:05
@jsolly
jsolly merged commit b271103 into main Oct 3, 2026
2 checks passed
@jsolly
jsolly deleted the codex/adopt-shadcn-svelte branch October 3, 2026 23:07
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