Opened 17 years ago

Closed 17 years ago

Last modified 12 months ago

#8085 closed patch

NewGUI: mouse wheel support

Reported by: eriktorbjorn Owned by: fingolfin
Priority: normal Component: GUI
Keywords: Cc:
Game:

Description

This is an attempt to add mouse wheel support to the
NewGUI list and scrollbar widgets. (I haven't looked
much at the NewGUI code before. I hope I didn't mess it
up too badly. :-)

Of course, this means the backend has to be updated
slightly so that it will generate EVENT_WHEELUP and
EVENT_WHEELDOWN events. I only know enough to do this
for the SDL backend, and only for SDL 1.2.5 and later
since it uses the SDL_BUTTON_WHEELUP and
SDL_BUTTON_WHEELDOWN constants.

It's probably easy to support earlier SDL versions, but
I didn't see much point in doing so.

Ticket imported from: #620627. Ticket imported from: patches/190.

Attachments (1)

mousewheel.diff (7.2 KB ) - added by eriktorbjorn 17 years ago.
Patch against an October 8 CVS snapshot

Download all attachments as: .zip

Change History (6)

by eriktorbjorn, 17 years ago

Attachment: mousewheel.diff added

Patch against an October 8 CVS snapshot

comment:1 by SF/ender, 17 years ago

Although the recommended version for ScummVM is always the latest, we
do need to support at least down to 1.2.3, for compatability reasons.

A lot of distributions obviously don't ship with the greatest and latest yet,
and theres no reason to require people to upgrade for a small feature that
many probably won't use.

comment:2 by eriktorbjorn, 17 years ago

Understood.

Note however that it should still be possible to compile
against older SDL libraries. (I haven't actually tried it,
but it was my intention.) It's just that it will never
generate mouse wheel events if you do.

What I meant by not supporting older SDL versions was that
maybe it would be possible to get it to work simply by
defining the SDL_BUTTON_WHEELUP and SDL_BUTTON_WHEELDOWN
constants. At least as far as X is concerned the mouse wheel
is just sending mouse button clicks. But why bother?

comment:3 by fingolfin, 17 years ago

BTW while there is no reason to enforce SDL 1.2.5, I'd still very strongly
recommend using it over any older version to everybody, as it fixes
many many bugs on Linux/Windows/MacOS

comment:4 by fingolfin, 17 years ago

Owner: set to fingolfin
Status: newclosed

comment:5 by digitall, 12 months ago

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