The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jun 11, 2026 11:55pm

PR Summary

Low Risk
Small, conditional layout change in shared modal chrome with no auth or data impact; only full sizing behavior changes.

Overview
Fixes off-center size='full' modals when the sidebar is open by no longer applying the content-area paddingLeft offset on the modal centering wrapper.

ModalContent now only applies the workflow-aware sidebar/panel padding for sizes up through xl; full clears that inline style so wide (~95vw) dialogs center on the full viewport. Smaller modal sizes keep the existing content-area centering behavior.

Docs on the size prop were updated to describe this split. Affects consumers such as the logs execution snapshot and deploy expanded preview modals.

^{Reviewed by Cursor Bugbot for commit eb39a87. Configure here.}

@greptile

@cursor review

Greptile Summary

This PR fixes visual centering of size='full' modals (e.g., execution snapshot viewer, deploy modal) that were being pushed off-center by the sidebar paddingLeft offset applied to the container div. The single change in modal.tsx skips that padding whenever size === 'full', letting those wide modals center against the full viewport, while smaller sizes keep the existing sidebar/panel offset behaviour.

• Centering logic in ModalContent now branches on size: 'full' gets style={undefined} (no offset); all other sizes keep the existing paddingLeft calculation.
• A JSDoc comment was added to ModalContentProps documenting the two centering behaviours.

Confidence Score: 5/5

Safe to merge — the change is a four-line conditional that only affects size='full' modals and leaves all other sizes untouched.

The fix is minimal and well-scoped: removing the padding offset for full modals has no side-effects on other sizes. The two full consumers (execution snapshot modal and deploy modal) both benefit from the correction, and the JSDoc is accurate. No new logic paths, no state changes, no data mutations.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[ModalContent renders] --> B{size === 'full'?}
    B -- Yes --> C[style = undefined\nno padding offset\ncenter against full viewport]
    B -- No --> D{isWorkflowPage?}
    D -- Yes --> E["paddingLeft = calc(--sidebar-width - --panel-width)"]
    D -- No --> F[paddingLeft = --sidebar-width]
    C --> G[Modal centered on 100vw]
    E --> H[Modal centered on content area\nworkflow page offset]
    F --> I[Modal centered on content area\nsidebar offset]

_{Reviews (1): Last reviewed commit: "fix(modal): center full-size modals agai..." | Re-trigger Greptile}

Greptile Summary

Fixes viewport-centering for full-size modals by skipping the sidebar-offset paddingLeft when size === 'full', allowing those modals to center against the full viewport. Smaller sizes (sm–xl) retain the existing content-area centering.

• The wrapping div now conditionally applies style={undefined} for full size, which is a clean React pattern equivalent to omitting the style prop entirely.
• The fix targets only the one place in modal.tsx where the offset is applied, so all size='full' consumers (execution snapshot, deploy modal) benefit automatically.

Confidence Score: 5/5

Safe to merge — the change is a four-line conditional that removes a CSS padding offset for one modal size variant and leaves all other paths untouched.

The diff is a single, well-scoped conditional: full-size modals no longer receive the sidebar-width left padding, so they center against the viewport as intended. The logic is easy to reason about, has no side effects on other sizes or pages, and aligns with how the 95vw modal was designed to span the viewport.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[ModalContent renders] --> B{size === 'full'?}
    B -- Yes --> C[style = undefined\nNo paddingLeft offset]
    B -- No --> D{isWorkflowPage?}
    D -- Yes --> E["paddingLeft = calc(sidebar-width - panel-width)"]
    D -- No --> F["paddingLeft = var(--sidebar-width)"]
    C --> G[Flex container centers modal\nagainst full viewport]
    E --> H[Flex container centers modal\nagainst content area]
    F --> H

_{Reviews (2): Last reviewed commit: "fix(modal): center full-size modals agai..." | Re-trigger Greptile}