fix(gui): keep models tabs aligned - #1423
Conversation
|
📝 WalkthroughWalkthroughThe Models workspace CSS now applies the 1200px ChangesModels workspace layout
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
✅ READY
Review readiness checklist
✅ 4/4 boxes ticked. This pull request is already Ready for Review. |
9191097 to
b28410b
Compare
|
Tip For best results, initiate chat on the files or code changes.
I am replying to the latest comment only. I do not have the full earlier comment chain. You are interacting with an AI system. |
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
|
|
Tip For best results, initiate chat on the files or code changes.
You are interacting with an AI system. |
|
Tip For best results, initiate chat on the files or code changes.
You are interacting with an AI system. |
Summary
1200pxcontent layout, while the Routing tab fell back to the default980pxlayout.Verification
./node_modules/.bin/bun run typecheck./node_modules/.bin/bun run privacy:scancd gui && ../node_modules/.bin/bun test tests/models-provider-head.test.ts(4 passed)cd gui && ../node_modules/.bin/bun --bun run lintcd gui && ../node_modules/.bin/bun run buildcd docs-site && ../node_modules/.bin/bun install --frozen-lockfile && ../node_modules/.bin/bun run build1294x714: switching between Models, Combo, and Routing keeps the catalog and routing content at the same horizontal position.bun run test: 10126 passed, 7 skipped, 3 unrelated existing/environment failures (two timeouts and one Codex shim assertion).Checklist
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
All CI tests are green on my local testing.
I pushed my PR to the latest dev commit.
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.
Summary by CodeRabbit
Bug Fixes
Tests