Animation: "Fit" und "Centre" #111

Closed
opened 2026-09-17 06:42:24 +02:00 by admin · 1 comment

Animation Tab: "Fit" Button tut nichts, kann raus, "Centre" macht das was Fit eigentlich macht, darum umbenennen in "Fit"

Animation Tab: "Fit" Button tut nichts, kann raus, "Centre" macht das was Fit eigentlich macht, darum umbenennen in "Fit"
admin added this to the Development project 2026-09-17 06:46:18 +02:00
admin moved this to Next in Development on 2026-09-17 06:46:31 +02:00
admin moved this to Done in Development on 2026-09-17 07:12:39 +02:00
Owner

Erledigt in 4c92a94.

Ursache: Jeder Asset-Tab bekommt in AssetTabEditor automatisch einen Fit-Knopf. Im Animation-Tab setzte der nur die Timeline auf Frame 0 und Standard-Zoom zurück — meist schon so, also sichtbar wirkungslos. Das eigentliche Einpassen der Vorschau steckte im Extra-Knopf Centre.

Fix:

  • AssetTabEditor: der Fit-Knopf ruft eine überschreibbare Methode fit(); alle anderen Tabs passen weiter ihren Canvas ein.
  • AnimationEditor: fit() passt die Vorschau ein, Centre ist entfernt, Fit ist nur mit gewählter Wurzel aktiv.
  • TimelineCanvas.fit() entfernt (ungenutzt), fit im TabCanvas-Interface optional.
  • Debug-Hook preview.centreAt() → preview.fitAt(), e2e 25 klickt den echten Fit-Knopf.

Tests:

  • Lokal: npm run check → 92 Files, 1191 Tests; GM_STEPS=24,25,26 → 101 Checks.
  • CI auf 4c92a94: Task 76 gate → CHECK EXIT 0, Task 77 suite → ALL PASSED – 943 checks, Smokes 24 / 19 / 21.

Wiki: Animations.md nennt Fit statt Centre (5113c10).

Erledigt in `4c92a94`. **Ursache:** Jeder Asset-Tab bekommt in `AssetTabEditor` automatisch einen **Fit**-Knopf. Im Animation-Tab setzte der nur die Timeline auf Frame 0 und Standard-Zoom zurück — meist schon so, also sichtbar wirkungslos. Das eigentliche Einpassen der Vorschau steckte im Extra-Knopf **Centre**. **Fix:** - `AssetTabEditor`: der Fit-Knopf ruft eine überschreibbare Methode `fit()`; alle anderen Tabs passen weiter ihren Canvas ein. - `AnimationEditor`: `fit()` passt die Vorschau ein, Centre ist entfernt, Fit ist nur mit gewählter Wurzel aktiv. - `TimelineCanvas.fit()` entfernt (ungenutzt), `fit` im `TabCanvas`-Interface optional. - Debug-Hook `preview.centreAt()` → `preview.fitAt()`, e2e 25 klickt den echten Fit-Knopf. **Tests:** - Lokal: `npm run check` → 92 Files, 1191 Tests; `GM_STEPS=24,25,26` → 101 Checks. - CI auf `4c92a94`: Task 76 `gate` → `CHECK EXIT 0`, Task 77 `suite` → `ALL PASSED – 943 checks`, Smokes 24 / 19 / 21. Wiki: `Animations.md` nennt Fit statt Centre (`5113c10`).
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: meleki/feuerwerk#111