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 619 - Inconsistency between windib and directx driver with "system key" behaviour
Summary: Inconsistency between windib and directx driver with "system key" behaviour
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: events (show other bugs)
Version: HG 1.2
Hardware: x86 Windows (All)
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
: 596 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-08-31 03:38 UTC by Tim Angus
Modified: 2009-04-13 01:45 UTC (History)
2 users (show)

See Also:

Attachments
Patch that disables "system key" handling with the directx driver (507 bytes, patch)
2008-08-31 03:39 UTC, Tim Angus 		Details | Diff
Same as previous patch, but applies cleanly on recent revisions (507 bytes, patch)
2009-04-02 08:33 UTC, Tim Angus 		Details | Diff
Show Obsolete (1) View All 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 Tim Angus 2008-08-31 03:38:36 UTC 
When pressing "system keys" e.g. ALT+F4 etc., the default for the windib driver is to ignore them completely, whereas with the directx driver they're handled (badly as it turns out).

As the windib driver is the default, I assume its behaviour is considered correct, so the following patch makes the directx driver the same in this regard.
Comment 1 Tim Angus 2008-08-31 03:39:43 UTC 
Created attachment 270 [details]
Patch that disables "system key" handling with the directx driver

The patch.
Comment 2 Tim Angus 2008-08-31 13:14:48 UTC 
*** Bug 596 has been marked as a duplicate of this bug. ***
Comment 3 Gregory Smith 2008-12-14 19:21:35 UTC 
Hmm, installing this patch results in the app receiving a tab key event when alt-tabbing out. This doesn't happen, as far as I can tell, with windib or pre-patch dx5.
Comment 4 Tim Angus 2009-04-02 08:33:42 UTC 
Created attachment 315 [details]
Same as previous patch, but applies cleanly on recent revisions
Comment 5 Sam Lantinga 2009-04-13 01:45:15 UTC 
This was committed with revision 4485.

Thanks!