Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category FM Tuner
  • Assigned To No-one
  • Operating System Sansa AMSv2
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by gree665 - 2012-03-14
Last edited by saratoga - 2012-11-25

FS#12616 - Clip+ Radio Keymap More Consistent

This makes the Radio screen buttons similar to the WPS buttons:
The differences:
-Home now brings you to root menu screen.
-Stop now stops playing and brings you to the root menu screen.
-Long Select toggles the scan/preset modes.

Closed by  saratoga
2012-11-25 21:10
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Accepted in f6d088c.

As far as I can see this patch was made using git. Please use gerrit for patches if possible. Also, at least I would appreciate the appropriate change to the manual as well.

(Can’t comment if the changed button mappings make sense / are useful since I don’t own a Clip)

I just figured out how to update the code with git, and I was looking for information about how to use gerrit, but I wasn’t sure if it is set up for people without commit status or not. Apparently it is, so I will look into it.

I don’t know how to make a patch for the manual, but the change needed is in: Section 5.9 FM Radio, in the the key/action table. The 3 that are changed marked with *(notes).

KEY ACTION
Left, Right Change frequency in SCAN mode or jump to next/previous station in PRESET mode.
Long Left, Long Right Seek to next station in SCAN mode.
Volume Up, Volume Down Change volume.
Power Stop the radio and return to Main Menu. *(now behaves similarly to WPS)
Home Leave the radio screen with the radio playing. *(now behaves similarly to WPS, bringing you to root menu, then back to Radio screen)
Play Mute radio playback.
Long Select Switch between SCAN and PRESET mode. *(used to be of home key)
Select Open a list of radio presets. You can view all the presets that you have, and switch to the station.
Submenu Display the FM radio settings menu.

Hmm Seems that submitting the comment messed up my columns there.

Great, thanks for the patch! It is really a mess to have a different keymaps in different screens.

Could you please map Long Select to the FM Radio Menu to be consistent with mapping to the Context Menu in other places?
Switching scan/preset modes could be probably long Up or long Down.

You’re right, long select should be the context menu for radio. I’ll make the change.

Ok I swapped the menu and the scan mode features. Here is the comparison with the WPS buttonmap:

ACTION WPS RADIO
Volume Up /Volume Down -Volume up/down -Volume up/down
Left/Right -Change track -Change station
Long Left/Right -Rewind/Fast forward. -Next station in SCAN mode.
Play -Toggle pause -Toggle mute
Long Play -Pitch screen -N/A
Power -Stop playback -Stop Radio (goes to main menu)
Select -File Browser/Database -Radio Presets
Long Select -Enter WPS Context Menu -Radio Menu
Home -Main Menu (still playing) -Main Menu (still playing)
Long Home -Switch to the Quick Screen -N/A
Home+Select Key -Toggle hold -N/A
Submenu -N/A -Switch between SCAN and PRESET
Long Submenu -Show current Playlist -N/A

I’ve update the above patch with manual changes. If no one minds, I’ll commit this version.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing