Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Left/right scrolling with touchpad in main game mode crashes UH #2753

Closed
frsfnrrg opened this issue Nov 10, 2017 · 2 comments
Closed

Left/right scrolling with touchpad in main game mode crashes UH #2753

frsfnrrg opened this issue Nov 10, 2017 · 2 comments
Milestone

Comments

@frsfnrrg
Copy link

On Arch Linux, having just installed unknown-horizons-git.

When I left or right scroll using the touchpad (xinput test # button presses 6 or 7),
if I am in the main menu or the New-Game Single Player screen, nothing happens
as soon as a game loads, L/R scrolling immediately exits the program with error

Traceback (most recent call last):
RuntimeError: Caught a director exception

Unknown Horizons exited uncleanly via SIGINT
AL lib: (EE) alc_cleanup: 1 device not closed

If I disable touchpad L/R scrolling, via xinput set-button-map ## 1 2 3 4 5 0 0
where ## corresponds to the touchpad, the crash does not occur.

@frsfnrrg frsfnrrg changed the title Crash when scrolling left or right in main game mode Crash when touchpad-scrolling left or right in main game mode Nov 10, 2017
@frsfnrrg frsfnrrg changed the title Crash when touchpad-scrolling left or right in main game mode Left/right scrolling with touchpad in main game mode crashes UH Nov 10, 2017
@LinuxDonald
Copy link
Member

Is this the only output from the crash?

@jmdejong
Copy link
Contributor

jmdejong commented Dec 16, 2018

This looks like the same problem as #2828 and fifengine/fifengine#1062, where horizontal scrolling would also crash the game
This would mean that it is fixed in #2829.

Short summary: UH was implementing a FIFE mouseeventlistener interface, but did not implement the methods for horizontal scrolling. Adding implementations for those fixed it.

I think we can close this.

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

No branches or pull requests

3 participants