We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 998 - Right mouse button doesn't work at all
Summary: Right mouse button doesn't work at all
Status: RESOLVED DUPLICATE of bug 910
Alias: None
Product: SDL
Classification: Unclassified
Component: events (show other bugs)
Version: HG 2.0
Hardware: x86 Mac OS X 10.6
: P2 major
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-06 14:04 UTC by Pavel Kanzelsberger
Modified: 2010-07-12 22:04 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Kanzelsberger 2010-05-06 14:04:41 UTC 
This can be always reproduced on any version of MacOSX.

I tried with 32bit Leopard 10.5 and 64bit Snow Leopard 10.6 with either MacBook Pro trackpad or attached bluetooth or USB mouse.

Right mouse button event will never enter the event queue. I quickly checked the source code and I wasn't able to find out what is wrong. Maybe some deprecated NSEvent method is being used and it reports correctly only for left mouse button.
Comment 1 Ryan C. Gordon 2010-05-06 22:06:30 UTC 
This is because this code was rewritten for SDL 1.3. It works in 1.2.

Jiang (our GSoC student from last year), had this thought in an IM conversation with me yesterday...

11:09:38 PM Jjgod Jiang: It seems the current mouse handling implementation cannot receive mouse events other than left mouse
11:10:38 PM Jjgod Jiang: I think that's because Cocoa_WindowListener inherits from NSResponder, while NSResponder normally does not handle these events (NSView will handle them directly, as the docs said.)

...but I haven't looked into it yet.

--ryan.
Comment 2 Sam Lantinga 2010-07-12 22:04:50 UTC 
Hey Ryan, there's a patch for this in bug 910, can you take a look?

*** This bug has been marked as a duplicate of bug 910 ***