Skip to content

fix(input): forward vertical mouse wheel to GUI - #74

Closed
kakra wants to merge 1 commit into
ralgar:mainfrom
kakra:fix/mouse-wheel
Closed

kakra wants to merge 1 commit into
ralgar:mainfrom
kakra:fix/mouse-wheel

Conversation

@kakra

@kakra kakra commented Aug 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Forward vertical mouse-wheel input from the XCB and Xlib backends to ImGui.

  • translate X11 button 4/5 presses into vertical wheel deltas
  • accumulate wheel input in InputManager
  • consume the accumulated delta once per GUI frame
  • observe Xlib wheel events through a separate display connection without consuming events from the application's queue
  • leave native Wayland axis normalization for a separately testable follow-up

Horizontal scrolling is intentionally outside the scope of this change.

Testing

  • release build with meson compile -C build
  • exercised through the vkShade GUI under ETS2 via Proton/XWayland

@kakra
kakra marked this pull request as ready for review August 24, 2026 01:33
@kakra
kakra force-pushed the fix/mouse-wheel branch 2 times, most recently from 960993b to fb1ba49 Compare August 25, 2026 20:02
@kakra
kakra marked this pull request as draft August 25, 2026 20:06
@kakra

kakra commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Converting back to draft to add more work:

  • horizontal mouse wheel support (I can test this here if the GUI has horizontal scroll areas)
  • Wayland support

@ralgar

ralgar commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Closing this now since it's superseded by #89

@ralgar ralgar closed this Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants