#448-G: GTK scroll wheel scrolls editor text in the wrong (inverted) direction

[JDonaghy]

Follow-up to the GTK→ShellApp event-dispatch migration. Parent: #448 (regression from #540).

Summary

Using the scroll wheel moves the editor text in the opposite direction from what's expected (and from pre-migration behavior): scrolling the wheel one way moves the viewport the other way.

Repro

1. Open a file taller than the viewport.
2. Scroll the mouse wheel down → text moves the wrong way (viewport goes up), and vice-versa.

Root-cause hint

The sign of delta_y when translating a GDK scroll event into quadraui::UiEvent::Scroll / ScrollDelta looks inverted on the new ShellApp path:

• src/gtk/mod.rs:1654 — Msg::MouseScroll { delta_x, delta_y } and the ScrollDelta::new(delta_x, delta_y) construction (~:1682)
• gdk_scroll_to_uievent in src/gtk/events.rs
• history: GTK palette: scrollwheel scrolls one row at a time (TUI amplifies) #191 (delta scaling)

Compare the polarity the ShellApp handle(UiEvent::Scroll) applies vs the old fn update path — one of them negates and the migration likely dropped/added a negation.

Acceptance

Scroll direction matches the pre-migration/native convention. Add a unit test on the GDK-scroll → UiEvent::Scroll translation asserting delta polarity.

[JDonaghy]

Coordinator Assignment

Machine: elitebook
Repo: vimcode
Files: (unspecified)
Do not touch: (nothing else in flight)

Briefing

Issue #554: #448-G: GTK scroll wheel scrolls editor text in the wrong (inverted) direction

Follow-up to the GTK→ShellApp event-dispatch migration. Parent: #448 (regression from #540).

Summary

Using the scroll wheel moves the editor text in the opposite direction from what's expected (and from pre-migration behavior): scrolling the wheel one way moves the viewport the other way.

Repro

1. Open a file taller than the viewport.
2. Scroll the mouse wheel down → text moves the wrong way (viewport goes up), and vice-versa.

Root-cause hint

The sign of delta_y when translating a GDK scroll event into quadraui::UiEvent::Scroll / ScrollDelta looks inverted on the new ShellApp path:

• src/gtk/mod.rs:1654 — Msg::MouseScroll { delta_x, delta_y } and the ScrollDelta::new(delta_x, delta_y) construction (~:1682)
• gdk_scroll_to_uievent in src/gtk/events.rs
• history: GTK palette: scrollwheel scrolls one row at a time (TUI amplifies) #191 (delta scaling)

Compare the polarity the ShellApp handle(UiEvent::Scroll) applies vs the old fn update path — one of them negates and the migration likely dropped/added a negation.

Acceptance

Scroll direction matches the pre-migration/native convention. Add a unit test on the GDK-scroll → UiEvent::Scroll translation asserting delta polarity.

[JDonaghy]

Coordinator: Assignment Failed

Machine: elitebook
Status: failed
Exit code: —
Duration: —
Log: /home/john/.coord/logs/2787d3768ed5.log

Error

pull failed for quadraui: git pull --ff-only failed: error: Unable to create '/home/john/src/quadraui/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

[JDonaghy]

Coordinator: Assignment Complete

Machine: precision
Status: done
Exit code: 0
Duration: 8m 18s
Log: /home/john/.coord/logs/f2b4aaf73c49.log