janpeter/Media-Organizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

IDEA-228: direct afspelen, voorkeuren onthouden en mapscroll herstellen #96

Merged
janpeter merged 5 commits from codex/idea-228-observaties into main 2026-09-28 07:08:59 +02:00
Conversation 1 Commits 5 Files changed 15 +713 -30
janpeter commented 2026-09-28 07:04:22 +02:00
Owner
Copy link

Een thumbnailklik opent de slideshow nu direct in afspeelstand. De keuzes voor 2× fotodiasnelheid en videobediening blijven in dezelfde browser bewaard na sluiten, mapwissel en herladen. Bij het kiezen van een andere map begint alleen het inhoudsscrollvak vóór paint bovenaan; bestaande boom-, pagina- en filterstaat blijft behouden.

Voorkeuren worden alleen bij expliciete bediening opgeslagen. Ontbrekende, ongeldige of geblokkeerde browseropslag heeft veilige defaults. Play/pause, selectie en fullscreen worden niet bewaard; 2× verandert de videosnelheid niet.

Verificatie:

  • Node 22.23.2 met eigen PostgreSQL17-testdatabase: 1.141 tests geslaagd, 0 fouten, 27 bestaande afzonderlijke DB/Linux-skips.
  • Gerichte hercontrole: 79/79 geslaagd; build en typecheck groen; lint 0 errors en 12 bestaande warnings.
  • Onafhankelijke review: GO na de correctie naar useLayoutEffect voor scrollreset vóór paint.
  • Chrome-proef met echte UI-componenten en synthetische media: direct afspelen, bewaren van aan én uit na reload/mapwissel en navigatie via boom, submap, Terug en zoeken gecontroleerd. Backendgrenzen waren expliciet gefixtured.

IDEA-228 · sprint S-2026-09-28-1 · ST-058 · T-215/T-216. Bewijs en beperkingen: docs/research/2026-09-28-idea-228-browserbewijs.md en docs/research/2026-09-28-idea-228-verificatie.json. Geen nieuwe dependency of databasewijziging. Geen merge of deployment uitgevoerd.

Een thumbnailklik opent de slideshow nu direct in afspeelstand. De keuzes voor 2× fotodiasnelheid en videobediening blijven in dezelfde browser bewaard na sluiten, mapwissel en herladen. Bij het kiezen van een andere map begint alleen het inhoudsscrollvak vóór paint bovenaan; bestaande boom-, pagina- en filterstaat blijft behouden. Voorkeuren worden alleen bij expliciete bediening opgeslagen. Ontbrekende, ongeldige of geblokkeerde browseropslag heeft veilige defaults. Play/pause, selectie en fullscreen worden niet bewaard; 2× verandert de videosnelheid niet. Verificatie: - Node 22.23.2 met eigen PostgreSQL17-testdatabase: 1.141 tests geslaagd, 0 fouten, 27 bestaande afzonderlijke DB/Linux-skips. - Gerichte hercontrole: 79/79 geslaagd; build en typecheck groen; lint 0 errors en 12 bestaande warnings. - Onafhankelijke review: GO na de correctie naar useLayoutEffect voor scrollreset vóór paint. - Chrome-proef met echte UI-componenten en synthetische media: direct afspelen, bewaren van aan én uit na reload/mapwissel en navigatie via boom, submap, Terug en zoeken gecontroleerd. Backendgrenzen waren expliciet gefixtured. IDEA-228 · sprint S-2026-09-28-1 · ST-058 · T-215/T-216. Bewijs en beperkingen: docs/research/2026-09-28-idea-228-browserbewijs.md en docs/research/2026-09-28-idea-228-verificatie.json. Geen nieuwe dependency of databasewijziging. Geen merge of deployment uitgevoerd.
docs: record IDEA-228 browser proof and final verification
All checks were successful
CI / test (historical-bootstrap) (pull_request) Successful in 1m5s
Details
CI / test (empty-trash) (pull_request) Successful in 8m32s
Details
CI / test (suite) (pull_request) Successful in 3m18s
Details
CI / test (video-migration) (pull_request) Successful in 48s
Details
CI / docker-build (pull_request) Successful in 1m55s
Details
7afd91b8b4
s4m-codex-reviewer approved these changes 2026-09-28 07:06:06 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

  • Geen findings met blocker-, error- of comment-severity aangetroffen.
  • Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
  • De wijziging houdt de voorkeuren beperkt tot de twee afgesproken booleanwaarden, behandelt browseropslag veilig, start afspelen na een thumbnailklik en reset uitsluitend het inhoudsscrollvak bij een daadwerkelijke mapwissel.
  • Gerichte regressietests voor voorkeuren en mapscroll zijn toegevoegd; de meegeleverde verificatiedocumentatie rapporteert typecheck, build en tests als geslaagd.
## Verdict: APPROVED - Geen findings met blocker-, error- of comment-severity aangetroffen. - Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. - De wijziging houdt de voorkeuren beperkt tot de twee afgesproken booleanwaarden, behandelt browseropslag veilig, start afspelen na een thumbnailklik en reset uitsluitend het inhoudsscrollvak bij een daadwerkelijke mapwissel. - Gerichte regressietests voor voorkeuren en mapscroll zijn toegevoegd; de meegeleverde verificatiedocumentatie rapporteert typecheck, build en tests als geslaagd.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Media-Organizer!96
No description provided.