Feat/polish-mission1 #10

Closed

math-pixel wants to merge 0 commits from feat/polish-mission1 into develop

pull from: feat/polish-mission1

merge into: math-pixel:develop

math-pixel:main

math-pixel:develop

math-pixel:feat/polish-mission-2

math-pixel:feat/e-bike

math-pixel:fix/repair-game

math-pixel:feat/polish-perf

math-pixel:feat/polisth-intro

math-pixel:feat/map-environment

math-pixel:feat/mission-2

math-pixel:design

math-pixel:feat/intro

math-pixel:feat/repair-game

Conversation 0 Commits - Files Changed -

+1754 -650

math-pixel commented 2026-05-30 00:16:27 +00:00

Owner

Copy Link

Copy Source

No description provided.

math-pixel added 1 commit 2026-05-30 00:16:27 +00:00

feat(types): add SiteStep and refactor GameStep for new intro flow 4c5e2ed945

math-pixel added 4 commits 2026-05-30 02:02:00 +00:00

update: reorganize public assets 8cfee1ac93

feat: add site onboarding route a2cff0567e

update: intro flow overlays ce5dc8ada0

feat(dialogues): support multi-cue subtitles 02c1fb33d0

math-pixel added 3 commits 2026-05-30 15:56:49 +00:00

fix(site): update situation cards 60e3c92511

fix(site): reduce situation card font size 29342d796c

fix(site): unified card styles, import Nersans One font, native naming input 6ae21a2427

math-pixel added 3 commits 2026-05-30 16:44:33 +00:00

refactor(site): extract shared utilities and centralise dialogue IDs ... 0f6860f1ae

- new src/hooks/ui/useIsMobile.ts (matchMedia + useSyncExternalStore)
  replacing the resize-handler hook inlined inside pages/site/page.tsx
- new src/hooks/ui/usePrefersReducedMotion.ts
- new src/data/site/dialogueIds.ts so site and intro components stop
  carrying hard-coded narrator IDs
- siteConfig: add SITE_BACKGROUND_STYLE shared by SiteLayout and
  SiteMobileBlocker, rename forcedName to presetPlayerName, fix the
  swapped id/label pairing on situation cards
- useSiteStore: rename selectedExperience/Situation to *Index so the
  stored value (an array index) is obvious in callers
- audioConfig: drop dead AUDIO_PATHS placeholders
- propagate the renames and SITE_BACKGROUND_STYLE through SiteLayout,
  SiteWelcomeScreen, SiteSituationScreen and pages/site/page.tsx

fix(site): repair onboarding audio cleanup, redirect, and manifest fetches ... 07b09c22af

- loadDialogueManifest: cache the resolved manifest at module level and
  dedupe concurrent fetches so each screen no longer re-downloads it
- useGameStore: completeIntroState now also advances intro.currentStep
  to "playing" so callers do not need a separate setIntroStep call
- SiteNamingScreen and SiteTransitionOverlay: replace ref-based guards
  with an isCancelled flag captured per effect. The previous guards
  persisted across StrictMode remounts, leaving mount 2 unable to
  re-run the effect after mount 1's chain was cancelled, which broke
  the fade animations, the second narrator dialogue and the redirect.
  Both screens now also call stopCurrentDialogue on unmount so audio
  cannot bleed across routes, and the transition gets a safety timeout
  in case the dialogue audio fails to fire its "ended" event
- SiteTransitionOverlay: keep the <Subtitles /> mount inside the
  overlay so it renders inside the z-index 1000 stacking context
  (above the black screen); the one in SiteLayout sits behind it
- IntroDialogueOverlay: route through playDialogueById instead of
  AudioManager.playSoundWithCallback so the narrator subtitles play
  in sync, and add the same isCancelled cleanup pattern
- IntroRevealOverlay: rely on completeIntro alone now that it advances
  intro.currentStep, and skip the fade when reduced motion is requested
- SiteMobileBlocker: correct logo path from public/... to /...

feat(a11y): WCAG AA polish on the site onboarding flow ... 970adf4853

- index.css: add visible :focus-visible rings for .site-card-button
  and .site-button so keyboard users can see where focus lives
- SiteCard: drop outline:none, add aria-pressed and aria-label so
  screen readers announce selection state
- SiteButton: add the .site-button class for the shared focus ring
- SiteDisclaimerScreen: keyboard skip via Enter / Space / Escape, a
  role="region" + aria-label wrapper and aria-live="polite" on the
  message; honour prefers-reduced-motion on the fade
- IntroVideoPlayer: role="region" with a skip hint in aria-label,
  preload="auto", and aria-hidden on the decorative caption span

math-pixel added 5 commits 2026-05-30 18:25:29 +00:00

fix(assets): correct texture URIs in gant_r and pylone GLTFs ... 82dc47a296

- gant_r/model.gltf: align casing with the actual files on disk
  (gant_basecolor / gant_occlusionroughnessmetallic) so the GLTFLoader
  stops logging 'Couldn't load texture' on case-sensitive filesystems
- pylone/model.gltf: nine missing Image_N.png references replaced with
  the existing color.png / normal.png so the pylone renders without
  console errors. Material slots are mapped by role: normalTexture ->
  normal.png, baseColorTexture and metallicRoughnessTexture -> color.png

fix(perf): prevent Canvas double-mount on /site redirect ... 0fa7a82175

HomePage used to mount the Canvas before its effect fired the redirect
to /site, then unmount it as soon as the route changed. That left the
WebGL context torn down mid-load with GLTF requests still in flight,
which on slow GPUs ended in a 'Context Lost' and a stuck 1 FPS render
once the user came back from /site. The fix is a synchronous cookie
check after all hooks: if the user has not visited /site today we
return null and let the redirect happen without ever creating a GL
context.

Also drops the GameMap 'lite map skipped' log from warn to info: it
is an expected lite-loading path, not a problem worth a yellow warning.

feat(intro): polish loading transition e6bfcbe960

chore(logging): downgrade 'lite map skipped' to debug ... f24704091a

This log fires every time the lite map loader skips heavy nodes, which
is the expected fast-path. It does not need to show up in a normal
console session — moving it to logger.debug keeps it accessible under
?debug for diagnostics while removing the noise from default runs.

fix: show dialogue subtitles on black screen 7c691a8044

math-pixel added 2 commits 2026-05-30 19:00:33 +00:00

Update SiteTransitionOverlay.tsx b578e68c2e

fix(webgl): auto-restore context after loss ... a3f611e227

The Canvas onCreated callback used to log Context Lost but never asked
the GPU to restore it, which left the page on a frozen black canvas
until the user reloaded. We now grab the WEBGL_lose_context extension
on mount and call restoreContext() 500ms after a loss, giving the GPU
time to free memory before we ask for a new context. The existing
webglcontextrestored handler reinstates the shadow map settings, so
recovery is transparent to the user.

This does not prevent context loss itself — frequent losses still
indicate VRAM pressure or HMR-driven context churn — but it removes
the need to reload manually when the GPU recycles us.

math-pixel closed this pull request 2026-05-31 08:43:10 +00:00

math-pixel deleted branch feat/polish-mission1 2026-05-31 08:43:10 +00:00

math-pixel referenced this issue from a commit 2026-06-03 06:42:45 +00:00

Merge pull request #10 from La-Fabrik-Durable/feat-animation

Some checks are pending

Hide all checks

🔍 Lint / 🪄 Check lint (pull_request) Has been cancelled

Details

🔍 Lint / 🎨 Check format (pull_request) Has been cancelled

Details

🔍 Lint / 🔎 Typecheck (pull_request) Has been cancelled

Details

📊 Quality / 🔒 Security Audit (pull_request) Has been cancelled

Details

📊 Quality / 📋 Dependency Freshness (pull_request) Has been cancelled

Details

📊 Quality / 📦 Bundle Size (pull_request) Has been cancelled

Details

🔍 Lint / 🏗 Build (pull_request) Has been cancelled

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

math-pixel

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: math-pixel/La-Fabrik#10