Repository navigation
fix(web): publish dialog's failure banner uses the shared error alert - #33893
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — The publish failure banner now uses the shared error alert styling while retaining its existing message, display condition, and accessibility role. Publishing behavior and repository data are unchanged. No code changes detected at You can add or adjust custom eligibility rules. Learn more. |
3851bcf to
51a4542
Compare
105d89f to
7683547
Compare
1f0f37f to
b0c2282
Compare
|
Note Written by Update: merged the rebased base
Verified in a real dev environmentHead
Recording on this PR: open the menu, fill the fields, Publish, see the failure alert, Cancel, then reopen to a clean form. Not verified
|
ae02238 to
34d194a
Compare
4b8b08e to
41b823c
Compare
34d194a to
d36d9a3
Compare
Co-Authored-By: Claude Opus 5.5 <[email protected]>
41b823c to
ce27509
Compare
e5616fc
into
pingdotgg:maria/redesign/4-settings-setup
…pingdotgg#33893) Co-authored-by: Claude Opus 5.5 <[email protected]>
…pingdotgg#33893) Co-authored-by: Claude Opus 5.5 <[email protected]>
…pingdotgg#33893) Co-authored-by: Claude Opus 5.5 <[email protected]>




Note
🤖 Claude Opus 5.5 on behalf of Shivam
On layer 4, the Publish repository dialog draws its "Publish failed" banner by hand: its own border, fill, padding and red text. That duplicates the shared alert surface. Macroscope's UI Consistency review pointed this out on #28842.
The banner now uses
Alert variant="error"withAlertTitleandAlertDescriptionfromui/alert, the same wayAddEnvironmentDialogandThemeImportDialogshow errors in a dialog body.Alertstill setsrole="alert", and the title and message are unchanged.Stacks on
maria/redesign/4-settings-setup.Same dev server, same dialog: a project with no remote, GitHub made unreachable from the server (isolated
ghconfig, dummy token, outbound HTTPS sent to a dead proxy), ownerdemo-owner, repositorypublish-demo, Publish clicked.Before (layer 4): the hand-built banner.

After: the shared error alert.

vp run check:changedagainst layer 4 at 105d89f: format, lint, knip, typecheck and the React compiler check passed. No tests changed; this is a visual change, shown above.The Agent context budget check fails on this PR because of layer 4, not this diff, which leaves
agentContextBudget.jsonalone: main's budget shrank in #26570 after layer 4's last run, and layer 4's file is 2, 2 and 16 bytes over it now.Claude Opus 5.5, Claude Code via T3 Code.
🤖 Generated with Claude Code