Bug 9031 - Shuffle tooltip undefined
: Shuffle tooltip undefined
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.1
: PC Other
: -- normal with 1 vote (vote)
: 7.x
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-04 14:29 UTC by KDF
Modified: 2009-07-31 10: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 KDF 2008-08-04 14:29:23 UTC
It's the result of a typo (which I'll fix shortly):

Index: html/SqueezeJS/UI.js
===================================================================
--- html/SqueezeJS/UI.js	(revision 22326)
+++ html/SqueezeJS/UI.js	(working copy)
@@ -885,8 +885,8 @@
 	updateState: function(newState){
 		this.state = newState || 0;
 		this.setIcon('');
-		this.setTooltip(SqueezeJS.string('shuffle' + ' - ' + SqueezeJS.string('shuffle' + this.state)));
-		this.setText(SqueezeJS.string('shuffle' + ' - ' + SqueezeJS.string('shuffle' + this.state)));
+		this.setTooltip(SqueezeJS.string('shuffle') + ' - ' + SqueezeJS.string('shuffle' + this.state));
+		this.setText(SqueezeJS.string('shuffle') + ' - ' + SqueezeJS.string('shuffle' + this.state));
 		this.setClass('btn-shuffle-' + this.state);
 	}
 });
Comment 1 KDF 2008-08-04 14:34:59 UTC
fix committed to 7.1.1 at change 22362
Comment 2 Michael Herger 2008-08-04 14:37:57 UTC
Many thanks!
Comment 3 James Richardson 2008-08-12 15:06:53 UTC
Verified fix in 7.2-22530
Comment 4 Chris Owens 2009-07-31 10:26:30 UTC
Reduce number of active targets for SC