- Status Closed
- Percent Complete
- 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
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
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.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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.