fix(ui): unify branded error pages - #7057
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Workspace error surfaces ( The Electron offline fallback is restyled to match (wordmark header, Season Sans, chip-style primary/secondary actions); Reviewed by Cursor Bugbot for commit 4acb05e. Configure here. |
|
@cursor review |
Greptile SummaryThe PR consolidates web not-found and fatal-error surfaces around shared branded status-page components.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/components/status-page/status-page.tsx | Introduces reusable content and page-frame components for branded status surfaces. |
| apps/sim/app/global-error.tsx | Replaces the generic Next.js fatal-error page with the shared branded layout and reset action. |
| apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/error/index.tsx | Adopts the shared workspace error shell and removes an unused full-screen rendering branch. |
| apps/desktop/static/offline.html | Restyles the bundled offline recovery page with branded typography, wordmark, and Chip-like actions. |
| apps/desktop/electron-builder.yml | Adds the Season Sans font to packaged desktop application resources. |
| apps/desktop/e2e/smoke.spec.ts | Expands offline-page smoke coverage for branding, typography, copy, and action geometry. |
Reviews (4): Last reviewed commit: "fix(desktop): package offline font relia..." | Re-trigger Greptile
|
@cursor review |
|
@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 4acb05e. Configure here.
Summary
Testing
bun run lint:checkbun run check:auditsbun run type-checkbun run docs-manifest:checkbun run --cwd apps/desktop test(61 files, 1,184 tests)bun run --cwd apps/desktop test:e2e --grep "unreachable origin"