Skip to content

style(model-pricing): balance 5:5 layout and fix editor panel overlap - #6839

Open
xjdanielwhu wants to merge 1 commit into
QuantumNous:mainfrom
xjdanielwhu:contribute/model-pricing-5-5-layout
Open

style(model-pricing): balance 5:5 layout and fix editor panel overlap#6839
xjdanielwhu wants to merge 1 commit into
QuantumNous:mainfrom
xjdanielwhu:contribute/model-pricing-5-5-layout

Conversation

@xjdanielwhu

@xjdanielwhu xjdanielwhu commented Aug 14, 2026

Copy link
Copy Markdown

Summary

  • Split the model list and the pricing editor into a balanced 5:5 layout on md/xl breakpoints.
  • In the pricing editor panel, give the field area more room and narrow the fixed preview column (220–260px → 160–200px).
  • Prevent the price card grid from overflowing the scrollable area by using minmax(min(100%,400px),1fr).

Files changed

  • web/src/features/system-settings/models/model-ratio-visual-editor.tsx
  • web/src/features/system-settings/models/model-pricing-sheet.tsx

Verification

  • Verified at a 1280×720 viewport: left model list and right editor each take 50% width.
  • No overlap between the scrollable field area and the fixed preview column in the editor panel.

Summary by CodeRabbit

  • Style
    • Improved responsive layouts in the pricing editor for better use of available space.
    • Balanced the model table and pricing editor columns across screen sizes.
    • Narrowed preview areas and enabled pricing sections to fit more comfortably on smaller screens.

- split model list / pricing editor columns evenly at 5:5
- narrow the preview column in the pricing editor panel
- prevent price card grid from overflowing the scrollable area
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2be44ea5-6eb1-4a58-92f7-7a474bc1d9a9

📥 Commits

Reviewing files that changed from the base of the PR and between 58d4e9b and c64ce24.

📒 Files selected for processing (2)
  • web/src/features/system-settings/models/model-pricing-sheet.tsx
  • web/src/features/system-settings/models/model-ratio-visual-editor.tsx

Walkthrough

The pricing editor now uses revised responsive grid widths. The main editor receives more space, preview bounds are narrower, pricing lanes can shrink below 400px, and the ratio visual editor uses equal-width columns.

Changes

Responsive editor layouts

Layer / File(s) Summary
Responsive grid adjustments
web/src/features/system-settings/models/model-pricing-sheet.tsx, web/src/features/system-settings/models/model-ratio-visual-editor.tsx
The pricing editor allocates more width to editing, supports narrower pricing lanes, and uses equal-width table and editor columns at responsive breakpoints.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to c64ce

This change adjusts the model pricing editor layout and prevents panel overflow without introducing a known production or correctness risk; no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related PRs

Poem

A rabbit hops through columns bright,
More room for edits, lanes tucked tight.
The preview narrows, grids align,
Equal widths make layouts shine.
:) wiggles in the responsive design.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the model pricing layout change and the editor panel overlap fix described in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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