Opened 17 years ago

Closed 17 years ago

Last modified 6 years ago

#3131 closed defect (fixed)

SDL: mouse emulation

Reported by: SF/redzebra Owned by: fingolfin
Priority: normal Component: Documentation
Version: Keywords:
Cc: Game:


problem: -------- 2 small code blocks were removed from backends/platform/sdl/events.cpp in rev 16120 and disable the mouse emulation mode.(2004-12-18)

The readme and site documentation however still claim the keyboard arrow keys simulate mouse movements.

suggestions: ------------ There might be cases or games where this is not prefereable by default but allowing it to be enabled at runtime by configuration or key-combination would be helpfull.

If that's a problem, a compile time option (--enable-mouse-emulation) would be sufficient

If neither is acceptable please update the distributed readme and site documentation.

note that simply restoring the 2 small original code pieces still works :-)

if noone wants to spend some time on this but is willing to state what would be a non intrusive or acceptable way to support this feature, I 'm willing to provide an tested patch.


Ticket imported from: #1687663. Ticket imported from: bugs/3131.

Change History (3)

comment:1 by fingolfin, 17 years ago

Thanks for highlighting the mismatch between README and reality, I updated the README.

The mouse emulation code was removed as it caused severe problems with several games which used the arrow keys for other purposes. After some discussion it was determined that this code is not really helpful for most people.

I am reluctant to re-add this even as optional code. IMO adding optional things should only be done when it's really necessary, but not to avoid making a clear decision.

If you think that this would be a good feature to have, please file a feature request, and explain why you think the world :) would be a better place with this enabled (i.e. just saying that it "would be helpful" is a bit thin -- how would it be helpful, and to whom?).

comment:2 by fingolfin, 17 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:3 by digitall, 6 years ago

Component: Documentation
Note: See TracTickets for help on using tickets.