- Status Closed
- Percent Complete
- Task Type Patches
- Category Settings
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#7980 - Select default option in an option list
This patch adds an action to the option_screen to cause the default value
for a particular setting to become the current selection. It allows you
to see which value is the default, and at that point you can confirm it
or not. The code to do this is all in there, it just needs to be moved to
separate functions so we can reuse it.
Particularly useful for settings with long ranges of options, like a lot
of the sound settings such as base and treble for example. I’m blind, and
I find it takes a lot of button’ing around to put the setting back to
zero. It’s also great for when music is playing and the sound menus are
not spoken. And as a bonus, it lets one see what the safe default option
is supposed to be when browsing an obscure option.
Do others think this is useful?
The only tough part is the button assignment of course. This patch has
button mappings for x5 and e200 only, and at this point I’ve just added
them to the list context, but perhaps a separate context could be
created. For X5 I chose SELECT+POWER. (I thought the awkwardness wasn’t
a big issue in this case.) For e200 it’s POWER+SELECT.
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
I like the idea, but not really sure how useful it would be.
also, instead of adding a new action, I tinhk ACTION_STD_CONTEXT would work fine in the option screen for reset as it shouldnt be used currently anyway.
Sdoyon:
I know you prefer me to not modify your patches.
But is it ok that I added a keymap for h300(and h100 I think)?
Sorry if you didn’t want this.
Hopefully I didn’t do anything wrong.
Anyway the keymap for h300 is play and rec.
Not the easiest key press but should work for this.
Here it is…
Oh! Don’t know how I missed that. Well then that makes the patch
really simple!
Version using ACTION_STD_CONTEXT to trigger the reset, so no need
to modify keymaps.
Cool. I will try it out. I used it last might on my player and it was great.
Thanks.
BTW Were my changes ok? Or was the indenting and stuff wrong. I know you have a better way now but just wanting feedback mainly on what I did wrong and can improve on.
I know I only added one line.
Thx.
Well… you had an extra space between ACTION_SETTINGS_RESET
and BUTTON_ON|BUTTON_REC. They normally have the columns aligned in a
nice table format. I know that’s hard to get right with speech though.
Sync.
Any objections to committing this?
Oops, That was an old version. Here it is. Sorry.
I still find this somewhat useful. So here’s a sync, and a commit
warning: unless there are objections, I’ll commit this in a couple of days.
looks ok to me..
except, in setting_reset() the string resseting is wrong.. it should be using the length struct member, not MAX_FILENAME (although, admitadly thats my mistake to being with.. but please fix in the commit)
Sure. Like this?
Looks OK to me.
And me.