Skip to content

fix(web): restore padding in provider sign-in setup box - #5

Merged
adampeterhiggins merged 1 commit into
mainfrom
t3code/fix-provider-setup-styling
Sep 17, 2026
Merged

adampeterhiggins merged 1 commit into
mainfrom
t3code/fix-provider-setup-styling

Conversation

@adampeterhiggins

Copy link
Copy Markdown
Owner

Summary

  • Upstream fix(web): align provider settings with shared settings rows pingdotgg/t3code#10571 aligned provider settings with shared settings rows: ProviderSetupSection was converted to SettingsRows (which carry their own px-3 sm:px-4 py-3 padding), and the <div className="px-3 py-3 sm:px-4"> wrapper around ProviderInstanceCard's setup slot was removed.
  • The fork's ProviderAuthSection still rendered free-form content that relied on that wrapper, so after the merge its content sat flush against the Setup box edges (see before shot).
  • Converts ProviderAuthSection to the same SettingsRow structure as ProviderSetupSection: a sign-in row carrying the status line plus flow/sign-out controls, a sign-in method row holding the picker or pasted-credential form, and a readout row for the recorded method. The section now owns its own padding and matches the Antigravity setup rows in the same card.

Before / after

Signed in:

before

after — signed in

Signed out / disabled:

after — picker

Test plan

  • Verified in a real web client (vp run dev + seeded state): signed-in state shows padded rows with right-aligned Sign out; disabled state shows the enable block, status, and method picker rows
  • vp run --filter web typecheck, vp lint, vp fmt --check on the touched file
  • Verify the paste-credential form and waiting-flow (link expiry / callback input) visually on a provider that offers them

Generated with Devin

Upstream pingdotgg#10571 aligned provider settings with shared settings rows:
ProviderSetupSection was converted to SettingsRows, which carry their
own padding, so the <div className="px-3 py-3 sm:px-4"> wrapper around
the card's setup slot was removed. ProviderAuthSection still rendered
free-form content relying on that wrapper, so after the merge the
sign-in section sat flush against the Setup box edges.

Convert ProviderAuthSection to the same SettingsRow structure:
a sign-in row carrying the status line and the flow/sign-out controls,
a sign-in method row holding the picker or the pasted-credential form,
and a readout row for the recorded method, matching
ProviderSetupSection's layout in the same card.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@adampeterhiggins
adampeterhiggins merged commit b61f399 into main Sep 17, 2026
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