Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category User Interface
  • Assigned To No-one
  • Operating System Iriver H300 series
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Version 3.0
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jteh - 2006-04-09
Last edited by lostlogic - 2006-04-10

FS#5074 - voice UI does not speak initially selected menu choice

When using an option selection screen (such as in the settings menu) while playing audio, the initially selected option is not spoken. For example, if one enters menu→Playback→Repeat, no option is initially spoken. However, if one moves through the options, they are spoken. The work around is to simply press up and then down (or vice versa) to determine the initial selection, but this is a little tedious. Note that this works correctly while audio is playing.

I was snooping around the code and noticed that this should in fact be happening. In set_option in settings.c:
bool set_option(const char* string, void* variable, enum optiontype type,

              const struct opt_items* options, int numoptions, void (*function)(int))

{

  ...
  gui_syncselect_draw(&select);
  if (global_settings.talk_menu)
      talk_id(options[select.options.option].voice_id, true);

I noticed that options using unit values (such as Sound Settings→Volume) do speak the initial setting. Looking at the code for talk_unit, I notice that it always sets enqueue to false. I have no idea why setting enqueue to true should fail for the voicing of the initial selection with set_option (talk_unit successfully speaks the unit type with enqueue set to false, as it should), but this seems to be the only difference I can spot. I am also curious as to why enqueue is set to true for the voicing of the initial selection with set_option anyway, as I would have thought one would want to have the previous voicing shut up when entering an option selection.

Closed by  linusnielsen
2006-04-26 12:58
Reason for closing:  Fixed
jteh commented on 2006-04-23 12:15

This has been fixed in the rework of the voice UI in cvs. Thanks!

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing