#3050 closed defect (fixed)
BASS: mouse initialized wrong in control panel
| Reported by: | salty-horse | Owned by: | fingolfin |
|---|---|---|---|
| Priority: | normal | Component: | Engine: Sky |
| Version: | Keywords: | ||
| Cc: | Game: | Beneath a Steel Sky |
Description
Using latest svn.
1) Enter the control panel. 2) Position the cursor over a button. 3) Press ESC and enter the control panel again. 4) The cursor is over a button, but clicking has no effect.
When the control panel is initialized, The current mouse x/y positions are taken from the mouse class. These values are out-dated since the mouse class last got them when the game started - the Sky class seems to manage its own x/y values.
I'm not familiar with the mouse class' purpose, but perhaps this problem is a cause for other bugs?
Ticket imported from: #1656880. Ticket imported from: bugs/3050.
Change History (4)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:3 by , 18 years ago
Actually, the cursor position stored in the Mouse class is updated regulary by SkyEngine via Mouse::mouseEngine. And doing this makes sure the mouse position is synced during each "frame" of the game.
The real problem is that the control panel uses its own even loop, and its own copy of _mouseX / _mouseY. And it never updates the SkyMouse nor the SkyEngine class with these values. Hence, when you press ESC, both SkyEngine and SkyMouse contain invalid values. When you then press F5, their old invalid values are passed to the "Control" class. Hence, if you move the mouse 1 pixel between ESC and F5, things work just fine.
So I updated class Control to notify the mouse class about mouse movements, which fixes the issue.
comment:4 by , 6 years ago
| Component: | → Engine: Sky |
|---|---|
| Game: | → Beneath a Steel Sky |
Well, it's not as bad as it sounds, you can just move the mouse and you can click on the button again