improvement(logs): increase log details panel max width from 40vw to 60vw#4449
improvement(logs): increase log details panel max width from 40vw to 60vw#4449waleedlatif1 merged 2 commits intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Documentation comments are updated to reflect the new 60% viewport-width limit. Reviewed by Cursor Bugbot for commit 79facfc. Configure here. |
Greptile SummaryThis PR increases the maximum draggable width of the log details side panel from 40vw to 60vw by changing
Confidence Score: 5/5Single-constant change with no logic modifications — safe to merge. The change touches only one constant and its accompanying documentation. All three JSDoc references are now consistent with the new 60vw cap, and the clamping logic that consumes the constant is unchanged. There are no new code paths, no state mutations, and no risk of regressions. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User drags panel] --> B[clampPanelWidth called]
B --> C[getMaxLogDetailsWidth]
C --> D{window defined?}
D -- Yes --> E[window.innerWidth × 0.6]
D -- No / SSR --> F[fallback: 1040px]
E --> G[max]
F --> G
G --> H[min = Math.min 320px, max]
H --> I[Math.max min, Math.min width, max]
I --> J[Clamped panel width ≤ 60vw]
Reviews (2): Last reviewed commit: "chore(logs): fix stale 65vw references i..." | Re-trigger Greptile |
|
Fixed in 79facfc — both stale JSDoc references updated from 65vw/65% to 60vw/60%. |
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 79facfc. Configure here.
Summary
Type of Change
Testing
Tested manually
Checklist