FS#10618 - CowonD2 EQ Menu Buttons

Attached to Project: Rockbox
Opened by Carsten Schreiter (Kuma76) - Friday, 25 September 2009, 15:34 GMT
Last edited by Rob Purchase (shotofadds) - Sunday, 11 October 2009, 15:29 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System Another
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


This patch changes the button assignment in the graphical EQ menu on a Cowon D2 like below and makes the EQ menu easier to use (of course in the 3x3 grid mode only).

TOP_MIDDLE, MIDDLE_RIGHT, BOTTOM_MIDDLE, MIDDLE_LEFT are to navigate through the EQ settings.

BOTTOM_LEFT and BOTTOM_RIGHT are to change settings (in that way the setting is visible when changing it)

TOP_LEFT is to leave the EQ menu.
This task depends upon

Closed by  Rob Purchase (shotofadds)
Sunday, 11 October 2009, 15:29 GMT
Reason for closing:  Out of Date
Additional comments about closing:  Part committed, part superceded by 10639
Comment by Thomas Martitz (kugel.) - Friday, 25 September 2009, 17:37 GMT
Can you instead fix the eq screen to work with the touchscreen mode? We're not really happy about the grid mode at all and rather see that removed in the long tern,
Comment by Rob Purchase (shotofadds) - Saturday, 26 September 2009, 11:25 GMT
I had a look into this last night, thinking there must be a much simpler way to get this working than introducing a load of #ifdef's. It turns out all the problems with the EQ screen were due to conflicts (duplicated mappings) in the existing touchscreen keymap.

The following patch makes it work properly using the existing controls (up, down, left, right, centre to change mode). I think it's somewhat cleaner.

(ps. kugel is right, this screen really needs some re-work for the "real" touchscreen mode - so I'll leave this task open after I commit the above)

EDIT: I removed too much, patch updated ;-)
   eq.diff (1.7 KiB)