Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • 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 ddalton - 2007-09-12
Last edited by ddalton - 2007-10-20

FS#7764 - Voice a couple more splash screens

This patch voices some extra splash screens. Like under the info screen if you scan the disk it says "Scanning disk".

Could someone see if they can fix a few things?
-When hold is on and you press a key nothing is spoken
-When restarting playback nothing is voiced

Also can someone find out if the error messages will talk?
Please comment.
Thanks.

   talkmore.diff (17.9 KiB)
Closed by  ddalton
2007-10-20 08:40
Reason for closing:  Rejected
Additional comments about closing:  

Not needed.
Sdoyon's patches voice this info.
The stuff that is useful like the plugin errors can go in the talk plugins patch.
Also it is very unlikely that you will come across these error messages. So just a waste of ram I think…

nls commented on 2007-09-14 16:08

regarding the first issue, this is impossible on most (all?) targets because you can not read the buttons when the hold switch is engaged.

ddalton commented on 2007-09-14 22:47

Ok I will fix that later. Is the rest of the patch ok?

nls commented on 2007-09-15 16:08

The patch looks ok to me, just one minor thing. There's a missing " from the line in the lang file which reads "Press off to shutdown" (with the first " missing)
Other than that have you tested all those splashes to find out if they actually speak? There might be some hidden issues with voicing.
Another thing to take into consideration is if the added voice clips are valuable to the interface as some of our devices have a rather limited amount of memory available we shouldn't add clips that aren't useful.

ddalton commented on 2007-10-04 05:23

I think it will need updating. I will need to test it more and test all the splashes.
Also I will remove the hold stuff and see what else could be voiced.
I know the "shuffling" clip won't voice. Wasn't that splash removed from rockbox? The actual splash message that is displayed on the screen?

ddalton commented on 2007-10-04 12:00

Ok here is an update. It adds the missing quote to english.lang. I also removed the hold stuff. And maybe others not sure.
What else needs voicing? Blind users?

   talk-more-update.patch (17.2 KiB)
ddalton commented on 2007-10-04 12:02

Also what patch should the speaking of the plugin errors be in? Should they be in this one? Or should I move them to p7704?

ddalton commented on 2007-10-04 12:14

Ok removed some eq stuff and removed all the playlist_viewer stuff. sdoyon's patches voice this information so I have now got this to apply cleanly with his patches. Now I just need to test all the splash screens and then it should be ok to commit. If people can test it would be greatly appreciated.
Thanks.

   talk-more-update-2.diff (16 KiB)
ddalton commented on 2007-10-18 08:18

Ok I need to remove the stuff that is in sdoyon's patches.
I will do that when I have some time.

Also maybe say something when you switch hold on.
(Sdoyon's patch shuts down when hold is switch to on). So it must be possible to just speak one string like:
"hold on"
Hmmm probably not needed.

So what else doesn't talk in rockbox?
Things like:
voicing of more plugins can go in another patchh.

Also file browser improvements with voice are in another patch I am working on.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing