Bug 6395 - Need faster visual feedback when settings change
: Need faster visual feedback when settings change
Status: CLOSED FIXED
Product: SB Controller
Classification: Unclassified
Component: Settings
: unspecified
: PC RedHat Linux
: -- minor (vote)
: 7.0.1
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-12-18 00:27 UTC by Michael Herger
Modified: 2008-05-15 12:26 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Herger 2007-12-18 00:27:11 UTC
Changing the language takes 3+ seconds on the current Jive hardware. But there's no visual (or acoustic) feedback before the end of this moment. This can be quite confusing and cause the user to select an option again. Or think something's gone wrong. Would be nice if the selection on the screen could be changed before the background update is started.
Comment 1 Blackketter Dean 2007-12-18 10:20:29 UTC
If it really takes 3 seconds, we should put up a spinning full screen window.  The trick would be to get the localized "Changing Language" screen to appear in the target language.
Comment 2 Ben Klaas 2007-12-26 09:25:57 UTC
Fix this for 7.0, priority P2

This behavior is curious...could have sworn this used to not take this long.
Comment 3 Ben Klaas 2007-12-27 07:15:06 UTC
A quick profile of the methods used in setting the language showed that the bottleneck is in reading/parsing the strings.txt files.

r1266 has a partial fix for this by prefetching the global_strings.txt file once instead of once per applet.

This causes the response to be close to halved-- 1.5+ secs instead of 3+ seconds. Still not acceptable, but better...

Will look into adding a spinning full screen window during that delay.
Comment 4 Ben Klaas 2007-12-27 12:12:09 UTC
r1268 adds a spinning progress popup, but it's not displaying on the screen immediately, so this is still not quite right.
Comment 5 Richard Titmuss 2008-01-11 06:30:59 UTC
This should be fixable now using a co-routine Task. Ask me for info if you need it.
Comment 6 Ben Klaas 2008-01-11 07:27:29 UTC
I've not used Task before. If you can either tell me how it works or point me to somewhere else where it's in use, that would be helpful.
Comment 7 Richard Titmuss 2008-01-11 13:39:35 UTC
Also the text on the popup should be "Please wait"
Comment 8 Michael Herger 2008-01-13 23:08:22 UTC
Much better already - thanks!
Comment 9 Blackketter Dean 2008-01-18 10:38:17 UTC
Good enough for 7.0, ok?

Comment 10 Richard Titmuss 2008-03-12 04:53:31 UTC
Reset priority
Comment 11 Ben Klaas 2008-03-19 08:30:49 UTC
removing priorities from all 7.0.1 target bugs for re-prioritizing
Comment 12 Richard Titmuss 2008-04-10 15:15:40 UTC
Fixed in jive 7.0.1 r2187. Ben did most of the work, I just tidied it up for him :)
Comment 13 James Richardson 2008-05-06 13:33:02 UTC
Verified in 2409
Comment 14 James Richardson 2008-05-15 12:26:42 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html