Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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
Attached to Project: Rockbox
Opened by sdoyon - 2007-10-17
Last edited by sdoyon - 2008-04-20

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.

Closed by  sdoyon
2008-04-20 14:13
Reason for closing:  Accepted

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…

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.

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.

BTW Were my changes ok? Or was the indenting and stuff wrong.

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?

Project Manager

Looks OK to me.

And me.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing