fix(models): fix mobile overflow and hide cost bars on small screens#4125
fix(models): fix mobile overflow and hide cost bars on small screens#4125waleedlatif1 merged 1 commit intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview On small screens, the cost chart’s stacked bars are now hidden ( Reviewed by Cursor Bugbot for commit b0eccde. Configure here. |
Greptile SummaryThis PR fixes mobile overflow on the model comparison charts page by reducing the Confidence Score: 5/5Safe to merge — targeted responsive Tailwind fixes with no logic changes and no P0/P1 issues. All changes are additive Tailwind breakpoint adjustments. No data, state, or API logic is touched. The ContextWindowChart bar intentionally remains visible on mobile (it is a single percentage-width div inside a flex-1 min-w-0 container and does not overflow), while the more complex stacked cost bars are hidden to match the intended design. No rules violations detected. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "fix(models): fix mobile overflow and hid..." | Re-trigger Greptile |
Summary
w-[90px]) to prevent price text from overflowing the container on narrow phones (~320px)hidden sm:flex) to match production behavior — bars are only shown onsm+screenssm:ml-2.5) since there's no bar to offset from on small screenstext-[11px]) to ensure the longest prices fit without overflowType of Change
Testing
Tested manually on small phone viewport (~320px) — no overflow, matches production behavior
Checklist