Rockbox

Tasklist

FS#10488 - Doom - rotate directional keys when screen is rotated

Attached to Project: Rockbox
Opened by Michael Chicoine (mc2739) - Wednesday, 05 August 2009, 13:43 GMT
Last edited by Michael Chicoine (mc2739) - Saturday, 20 August 2011, 01:14 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch for the Doom plugin will rotate the directional keys to match the screen rotation.

Screen rotation in Doom is only available for targets where the screen height is greater than the screen width (e200, gigabeat f,x)

Also included is a patch for manual changes for the above and additional cleanup.
This task depends upon

Closed by  Michael Chicoine (mc2739)
Saturday, 20 August 2011, 01:14 GMT
Reason for closing:  Out of Date
Comment by Teruaki Kawashima (teru) - Wednesday, 05 August 2009, 14:58 GMT
IIRC, rotate_screen is only defined for targets with portrait LCD.
so, i guess your patch would break other targets than e200 and gigabeat.
patch for manual seems to add rotate option for ipodvideo for which i don't think screen rotation is supported.
or am i misunderstanding something?

this is my old patch tried to do same thing.
Comment by Michael Chicoine (mc2739) - Wednesday, 05 August 2009, 15:46 GMT
The key patch only changes the key mapping on targets that can rotate the display.

The manual patch is incorrect due to a copy/paste error. The corrected manual patch is attached.
Comment by Michael Chicoine (mc2739) - Wednesday, 05 August 2009, 16:03 GMT
Upon further review, you are correct, rotate_screen will not exist if the LCD is not portrait. This updated patch should not cause errors on other targets.
Comment by Michael Chicoine (mc2739) - Thursday, 06 August 2009, 05:32 GMT
Update again to correct cut/paste & "in too much of a hurry" errors

Loading...