fix(editor): scroll the animation list so every asset is reachable #71

Merged

marco merged 1 commit from feat/editor-list-scroll into main

2026-07-19 08:15:42 +02:00

marco commented

2026-07-19 08:12:48 +02:00

Owner

The Asset Editor drew all 46+ animations from a fixed top with no scroll, so everything past ~24 rows (the dummy, effects, statics, land, gump) fell off the bottom of the window — you only saw human/* and creature/critter/*.

Fix: mouse-wheel scrolling over a fixed visible window, with a N-M / total (scroll) hint; click hit-testing follows the scroll offset. Now every packed asset is reachable.

just lint → 0 warnings.

Follow-up: a proper namespace tree view + broader editor UI/UX pass is tracked in #70 (owner: "una vista ad albero degli assets").

Definition of Done

Every animation in the pack is reachable in the editor (scroll past the first screen).
Clicking a row selects the right asset regardless of scroll.
Build clean, zero warnings.

The Asset Editor drew all 46+ animations from a fixed top with **no scroll**, so everything past ~24 rows (the dummy, effects, statics, land, gump) fell off the bottom of the window — you only saw `human/*` and `creature/critter/*`. Fix: **mouse-wheel scrolling** over a fixed visible window, with a `N-M / total (scroll)` hint; click hit-testing follows the scroll offset. Now every packed asset is reachable. `just lint` → 0 warnings. Follow-up: a proper **namespace tree view** + broader editor UI/UX pass is tracked in #70 (owner: "una vista ad albero degli assets"). ## Definition of Done - [x] Every animation in the pack is reachable in the editor (scroll past the first screen). - [x] Clicking a row selects the right asset regardless of scroll. - [x] Build clean, zero warnings.

marco added 1 commit

2026-07-19 08:12:49 +02:00

fix(editor): scroll the animation list so every asset is reachable

ci / Lint & Test (pull_request) Successful in 1m48s

Details

4007b3073e

The list drew all 46+ animations from a fixed top with no scroll, so anything
past ~24 rows (dummy, effect, static, land, gump) fell off the bottom of the
window and was unreachable. Add mouse-wheel scrolling over a fixed visible
window, with a "N-M / total" hint; click hit-testing follows the scroll offset.
A namespace tree view + a broader UI/UX pass is tracked separately.

marco merged commit e9ab378cdf into main

2026-07-19 08:15:42 +02:00

marco deleted branch feat/editor-list-scroll

2026-07-19 08:15:42 +02:00

marco referenced this pull request from a commit

2026-07-19 08:15:44 +02:00

fix(editor): scroll the animation list so every asset is reachable (#71)