Skip to content

feat(startup): add DES-24 preloader and responsive mobile/error screens - #1847

Draft
paulfalgout wants to merge 6 commits into
developfrom
feature/des-24-preloader-mobile
Draft

paulfalgout wants to merge 6 commits into
developfrom
feature/des-24-preloader-mobile

Conversation

@paulfalgout

@paulfalgout paulfalgout commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Add animation from the first HTML frame, seamless runtime handoff, clearer interruption screens, and responsive search and dialogs.

DES-24 — production-ready startup animation

A visible start

Production loading sequence: the HTML visual animates before JavaScript, its logo and animation phase carry into the runtime, then synthetic account-setup guidance appears.

The initial HTML includes the shared glow, core and rings, so the visual animates before JavaScript or authentication completes. The runtime adopts the same visual and preserves its animation phase. Dismissal begins a 160 ms exit immediately, without a reveal delay or minimum-visible hold; it does not hold app readiness. Reduced-motion support remains.

Ready workspace switches now show content without fading in from opacity 0.

Captured from the production build with deliberately held requests and a synthetic 403 response. This shows appearance and handoff, not a startup-speed measurement or the normal-success exit.

Clearer interruptions

Before and after: the 404, 500 and account-setup screens share a dark ambient background, clearer typography and consistent branding.

404/500 retain their workspace Back action. “Hold up” keeps its account-setup guidance and has no new action. Organization sign-in keeps its existing design while removing its invisible delay; the unused prelogin spinner branch is removed.

Mobile interactions

Before and after mobile search and dialogs: search gains back/clear controls and stacked result metadata; full dialogs fill the viewport with actions at the bottom. All records are synthetic examples.

  • Full-screen search, back/clear controls and keyboard-accessible result buttons.
  • Full-height dialogs, compact confirmations, safe-area spacing and embedded-form visual-viewport listeners.
  • Dialog labels, focus containment/restoration and topmost-only Escape/native close handling. The global button-minimum override is removed; component-specific sizing remains.

Review focus

Four commits: startup/recovery → error/account setup → mobile/dialogs → workspace fade removal.

Verification

At 3d69acab, focused checks passed:

Check Result
Production build and lint Passed
Focused navigation/startup 28/28
Startup/recovery E2E · reduced motion 8/8
Component tests · previous e7981f4e 5/5

The full suite is incomplete. The previously observed broader non-JSON error failure also reproduced on its prior head. Live API/authentication, real mobile keyboard/safe areas, nested-dialog interactions and precise animation timing still need QA.

Visual evidence: the loading clip uses the production build at e7981f4e, with animation before JavaScript and phase-preserving runtime adoption; the loader is unchanged at 3d69acab. Status/mobile comparisons retain their 24 error-free captures at cc9b22fa; their owning source files are unchanged through 3d69acab. Their before images use pinned develop snapshot bf27195c, rather than the current develop tip. All content is synthetic. Held requests and a synthetic 403 demonstrate appearance and handoff, not measured speed, live authentication or normal-success exit; blocked service workers produced a recorded Workbox registration error.

QA1 now serves 3d69acab8d91784909e2cf3054f4041e655fb9f6.

@paulfalgout paulfalgout added the ci:defer-cypress Hold required Cypress jobs for approval during AI review iteration label Oct 16, 2026
@coderabbitai

coderabbitai Bot commented Oct 16, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@paulfalgout paulfalgout changed the title feat(startup): add DES-24 preloader and scoped mobile fixes feat(startup): add DES-24 preloader and responsive mobile/error screens Oct 16, 2026
@paulfalgout
paulfalgout force-pushed the feature/des-24-preloader-mobile branch 6 times, most recently from 3204bf3 to cc9b22f Compare October 16, 2026 14:14
@paulfalgout
paulfalgout force-pushed the feature/des-24-preloader-mobile branch 2 times, most recently from e7981f4 to 2640593 Compare October 16, 2026 15:33
@paulfalgout
paulfalgout force-pushed the feature/des-24-preloader-mobile branch from 2640593 to 3d69aca Compare October 16, 2026 15:35

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:defer-cypress Hold required Cypress jobs for approval during AI review iteration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant