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

M44 — Vis-design voor de Media-Organizer (PR D) #101

Merged
janpeter merged 11 commits from feat/m44-vis-design into main 2026-10-04 16:17:25 +02:00
Conversation 1 Commits 11 Files changed 45 +741 -207
janpeter commented 2026-10-04 15:21:03 +02:00
Owner
Copy link

De Media-Organizer krijgt het Vis-thema dat Scrum4Me en scrum4me-workers in M42 kregen: warm papier, inkt-tekst en één ember-accent voor de hoofdactie van een scherm.

Wat verandert

  • src/app/styles/theme.css is een bytegelijke kopie van app/styles/theme.css uit Scrum4Me (sha256 e01085a3…947dd22d); de grijze shadcn-waarden zijn uit globals.css.
  • Public Sans, Source Serif 4 en Geist Mono.
  • Knoppen in pilvorm; variant ember op inloggen, account aanmaken, mount toevoegen, wachtwoord wijzigen en koppelen (docs/m44-ember-lijst.md).
  • Invoervelden met border-control-border, onderstreepte tekstlinks, statusbadges op hun eigen tint.
  • De diavoorstelling blijft wit op zwart en draagt een vaste dark-klasse; de zes ruwe paletklassen gebruiken tokens. Prullenbak-actief is een gevuld pictogram.
  • Contrasttest uit Scrum4Me, omgezet naar node:test (761 gevallen).
  • docs/design/styling.md en een verwijzing in CLAUDE.md.

Alleen klassen, lettertypen en documentatie; geen gedragswijziging. De app blijft alleen licht.

Verificatie

  • npm run lint (0 fouten, 11 bestaande waarschuwingen), npm run typecheck en npm run build groen.
  • npm test met een lokale testdatabase: 1.903 geslaagd, 31 overgeslagen, 3 mislukt. De drie mislukte zijn "mounted … integration"-tests die in de uitvoer van een kindproces # pass N zoeken; onder Node 26 lokaal staat er ℹ pass N. De kindtests zelf slagen. Niet nagegaan of ze op main lokaal ook falen.
  • Paletcontrole over src: 0 ruwe paletklassen.
  • Routecontrole in de browser op een productiebuild met een lokale proefdatabase en gegenereerde testafbeeldingen: docs/m44-bewijs/increment-1/controle.md, met elf beelden. JP keurde de proef (increment 0) goed op 2026-10-04.

Niet gedaan

  • Op 375 px is alleen /login bekeken.
  • /setup, favorieten, verzamelingen, de tweede stap van koppelen en de videobediening zijn niet vastgelegd.

Plan: Scrum4Me docs/plans/M44-vis-ops-en-media.md. PBI-34, ST-063, ST-064, T-227–T-236.

🤖 Generated with Claude Code

De Media-Organizer krijgt het Vis-thema dat Scrum4Me en scrum4me-workers in M42 kregen: warm papier, inkt-tekst en één ember-accent voor de hoofdactie van een scherm. ## Wat verandert - `src/app/styles/theme.css` is een bytegelijke kopie van `app/styles/theme.css` uit Scrum4Me (sha256 `e01085a3…947dd22d`); de grijze shadcn-waarden zijn uit `globals.css`. - Public Sans, Source Serif 4 en Geist Mono. - Knoppen in pilvorm; variant `ember` op inloggen, account aanmaken, mount toevoegen, wachtwoord wijzigen en koppelen (`docs/m44-ember-lijst.md`). - Invoervelden met `border-control-border`, onderstreepte tekstlinks, statusbadges op hun eigen tint. - De diavoorstelling blijft wit op zwart en draagt een vaste `dark`-klasse; de zes ruwe paletklassen gebruiken tokens. Prullenbak-actief is een gevuld pictogram. - Contrasttest uit Scrum4Me, omgezet naar `node:test` (761 gevallen). - `docs/design/styling.md` en een verwijzing in `CLAUDE.md`. Alleen klassen, lettertypen en documentatie; geen gedragswijziging. De app blijft alleen licht. ## Verificatie - `npm run lint` (0 fouten, 11 bestaande waarschuwingen), `npm run typecheck` en `npm run build` groen. - `npm test` met een lokale testdatabase: 1.903 geslaagd, 31 overgeslagen, 3 mislukt. De drie mislukte zijn "mounted … integration"-tests die in de uitvoer van een kindproces `# pass N` zoeken; onder Node 26 lokaal staat er `ℹ pass N`. De kindtests zelf slagen. Niet nagegaan of ze op `main` lokaal ook falen. - Paletcontrole over `src`: 0 ruwe paletklassen. - Routecontrole in de browser op een productiebuild met een lokale proefdatabase en gegenereerde testafbeeldingen: `docs/m44-bewijs/increment-1/controle.md`, met elf beelden. JP keurde de proef (increment 0) goed op 2026-10-04. ## Niet gedaan - Op 375 px is alleen `/login` bekeken. - `/setup`, favorieten, verzamelingen, de tweede stap van koppelen en de videobediening zijn niet vastgelegd. Plan: Scrum4Me `docs/plans/M44-vis-ops-en-media.md`. PBI-34, ST-063, ST-064, T-227–T-236. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
src/app/styles/theme.css is bytegelijk aan app/styles/theme.css op Scrum4Me
origin/main (sha256 e01085a3432d6059810c2f8a691e55714216b38b0f67c165c2532980947dd22d,
beide). De grijze shadcn-waarden zijn uit globals.css; font-heading bestond
niet in het nieuwe thema en is font-serif geworden (kaart- en dialoogtitels).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
761 gevallen, gelijk aan Scrum4Me. Roodcontrole: --foreground op #cccccc
geeft 9 fouten; daarna teruggezet (theme.css weer bytegelijk).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Gelijk aan Scrum4Me components/ui/button.tsx: rounded-full als basis,
icon en icon-lg rounded-lg, destructive in donker 15/20%.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Statusbadges zonder dark-toevoeging, velden met border-control-border,
tekstlinks onderstreept met ember bij hover.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Ember op inloggen, account aanmaken, mount toevoegen, wachtwoord wijzigen
en de twee toestanden van koppelen; lijst in docs/m44-ember-lijst.md.
Tekstknoppen in pair, videogegevens en schijfverwijdering gebruiken de
knopcomponent; rij-, kaart- en overlayknoppen blijven button.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
De diavoorstelling krijgt een vaste dark-klasse; de zes ruwe paletklassen
(rood voor prullenbak, favoriet en foutmelding, amber voor videostatus)
gebruiken nu error, error-container en warning. Wit op zwart blijft.
Prullenbak-actief is nu een gevuld pictogram, zodat het ook zonder
kleur van niet-actief verschilt.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
docs(T-236): stylingdocument voor het Vis-thema en verwijzing in CLAUDE.md
All checks were successful
CI / test (historical-bootstrap) (pull_request) Successful in 1m8s
Details
CI / test (suite) (pull_request) Successful in 3m18s
Details
CI / test (video-migration) (pull_request) Successful in 1m5s
Details
CI / docker-build (pull_request) Successful in 2m42s
Details
CI / test (empty-trash) (pull_request) Successful in 10m21s
Details
bb3a994dd7 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
s4m-codex-reviewer reviewed 2026-10-04 15:29:35 +02:00
s4m-codex-reviewer left a comment
Copy link

COMMENT

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Geen concrete blokkerende codefout gevonden in de diff op commit bb3a994dd7bfcff0797072dc31eb547af2934777. De migratie hergebruikt semantische tokens en bestaande UI-primitieven; auth-, data- en verwijderlogica blijven in de onderzochte wijziging behouden.

Findings

  • warning — docs/design/styling.md:3 — De nieuwe Vis-richtlijn wijkt af van de actieve ProductDoc patterns/md3-theme-styling, die nog MD3-blauw, Geist en een light/dark-toggle voorschrijft. Werk de productstandaard bij of markeer deze als vervangen, zodat volgende agents niet twee tegenstrijdige stylingrichtlijnen volgen.
  • warning — docs/m44-bewijs/increment-1/controle.md:21 — Voor deze globale wijziging van fonts, knoppen en velden ontbreekt browserbewijs voor videobediening en de gevonden-aanvraagtoestand van /pair; op 375 px is alleen /login gecontroleerd. Controleer ook /account, /mounts en bibliotheekbediening op kleine schermen, inclusief focus, tekstoverloop en video-statuskleuren.

Verificatie

  • Exacte head-commit opgehaald; aangeleverde diff gecontroleerd tegen de repository.
  • Contrasttest: 761 geslaagd.
  • Gerichte tests voor Button, DiskDeleteControls, SlideshowActions en SlideshowVideoControls: 15 geslaagd, via een tijdelijke TypeScript-loader.
  • Typecheck: geslaagd na Prisma-generatie en ophalen van de gepinde submodules.
  • Lint: 0 fouten, 11 waarschuwingen. Diff-whitespacecheck geslaagd.
  • Productiebuild niet bevestigd: deze omgeving mount /tmp met noexec, waardoor native esbuild/SWC/Lightning CSS niet uitvoerbaar zijn. Volledige database-testsuite niet uitgevoerd. Het meegeleverde controleverslag rapporteert bovendien drie mislukte tests onder Node 26; dit is geen onafhankelijk bevestigde groene suite.

Door deze verificatiebeperkingen en de open documentatie-/browseropmerkingen kies ik COMMENT.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Geen concrete blokkerende codefout gevonden in de diff op commit `bb3a994dd7bfcff0797072dc31eb547af2934777`. De migratie hergebruikt semantische tokens en bestaande UI-primitieven; auth-, data- en verwijderlogica blijven in de onderzochte wijziging behouden. ## Findings - **warning — docs/design/styling.md:3** — De nieuwe Vis-richtlijn wijkt af van de actieve ProductDoc patterns/md3-theme-styling, die nog MD3-blauw, Geist en een light/dark-toggle voorschrijft. Werk de productstandaard bij of markeer deze als vervangen, zodat volgende agents niet twee tegenstrijdige stylingrichtlijnen volgen. - **warning — docs/m44-bewijs/increment-1/controle.md:21** — Voor deze globale wijziging van fonts, knoppen en velden ontbreekt browserbewijs voor videobediening en de gevonden-aanvraagtoestand van /pair; op 375 px is alleen /login gecontroleerd. Controleer ook /account, /mounts en bibliotheekbediening op kleine schermen, inclusief focus, tekstoverloop en video-statuskleuren. ## Verificatie - Exacte head-commit opgehaald; aangeleverde diff gecontroleerd tegen de repository. - Contrasttest: **761 geslaagd**. - Gerichte tests voor Button, DiskDeleteControls, SlideshowActions en SlideshowVideoControls: **15 geslaagd**, via een tijdelijke TypeScript-loader. - Typecheck: **geslaagd** na Prisma-generatie en ophalen van de gepinde submodules. - Lint: **0 fouten, 11 waarschuwingen**. Diff-whitespacecheck geslaagd. - Productiebuild niet bevestigd: deze omgeving mount `/tmp` met `noexec`, waardoor native esbuild/SWC/Lightning CSS niet uitvoerbaar zijn. Volledige database-testsuite niet uitgevoerd. Het meegeleverde controleverslag rapporteert bovendien drie mislukte tests onder Node 26; dit is geen onafhankelijk bevestigde groene suite. Door deze verificatiebeperkingen en de open documentatie-/browseropmerkingen kies ik COMMENT.
Sign in to join this conversation.
Reviewers
No reviewers
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!101
No description provided.