Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#8012 : voice a couple more things in the info list

Rockbox

Tasklist

FS#8012 - voice a couple more things in the info list

Attached to Project: Rockbox
Opened by Daniel Dalton (ddalton) - Monday, 22 October 2007, 07:14 GMT
Last edited by Jonathan Gordon (jdgordon) - Wednesday, 24 October 2007, 12:52 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch just voices the version (spells it). Also say the battery time in minutes/hours. Don't say minute or hour just say the number for example:
15 22. Is 15 hours 22 minutes.

Also speak the disk size because we can...

Problems:
-Two options aren't spoken on h300.
-Version probably shouldn't be first in the list. How do I change that? Maybe version last and battery level first.
I don't want to voice something different to what's on the screen too much.
Here is the patch anyway. Simular to  FS#6138  by Stéphane Doyon.

But that one is out of date due to JdGordon's changes to the info screen...
   talk-info-list.patch (2.4 KiB)
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Wednesday, 24 October 2007, 12:52 GMT
Reason for closing:  Accepted
Comment by Daniel Dalton (ddalton) - Monday, 22 October 2007, 07:18 GMT
Hmmm maybe one of the options is just blank. That's what I can tell from the code. And the other one might be for a second disk or something. Which I don't have.
Can someone confirm this?
Comment by Steve Bavin (pondlife) - Wednesday, 24 October 2007, 12:24 GMT
The attached patch works better for me. Also made the talk_ interface for time and date more consistent with the rest of the talk_ API
   infoscreen_2.patch (14.7 KiB)
Comment by Steve Bavin (pondlife) - Wednesday, 24 October 2007, 12:27 GMT
Just noticed the say_time() function, maybe this could be merged into talk_time()?

Loading...