[Slim-Checkins] r33664 - in /7.7/trunk/server: Changelog7.html HTML/EN/html/SqueezeJS/UI.js

Author: mherger
Date: Sun Nov  6 22:51:16 2011
New Revision: 33664

URL: http://svn.slimdevices.com/slim?rev=33664&view=rev
Log:
Fixed Bug: 17728
Description: don't trigger track info page when clicking buttons in playlist panel.

Tested in IE9, Safari, Chrome, Opera, FF

Please make sure you clear your caches etc. before testing this change. Thanks!

Modified:
    7.7/trunk/server/Changelog7.html
    7.7/trunk/server/HTML/EN/html/SqueezeJS/UI.js

Modified: 7.7/trunk/server/Changelog7.html
URL: http://svn.slimdevices.com/slim/7.7/trunk/server/Changelog7.html?rev=33664&r1=33663&r2=33664&view=diff
==============================================================================
--- 7.7/trunk/server/Changelog7.html (original)
+++ 7.7/trunk/server/Changelog7.html Sun Nov  6 22:51:16 2011
@@ -14,6 +14,7 @@
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17644">#17644</a> - Squeezebox database and logfiles fill up OS-partition</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17719">#17719</a> - Now Playing playlist in the web UI doesn't show correct metadata for remote music services</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17726">#17726</a> - Windows Control Panel can't authenticate to password protected server</li>
+		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17728">#17728</a> - Web GUI Jumps to Track More Info on Playlist Play</li>
 	</ul>
 	<br />
 </ul>

Modified: 7.7/trunk/server/HTML/EN/html/SqueezeJS/UI.js
URL: http://svn.slimdevices.com/slim/7.7/trunk/server/HTML/EN/html/SqueezeJS/UI.js?rev=33664&r1=33663&r2=33664&view=diff
==============================================================================
--- 7.7/trunk/server/HTML/EN/html/SqueezeJS/UI.js (original)
+++ 7.7/trunk/server/HTML/EN/html/SqueezeJS/UI.js Sun Nov  6 22:51:16 2011
@@ -841,7 +841,8 @@
 		if (target.hasClass('browseItemDetail') || target.hasClass('playlistSongDetail'))
 			target = Ext.get(target.findParentNode('div'));
 		
-		else if (target.dom.localName = 'img' && !target.findParentNode('span.browsedbControls', 3) && (target.findParentNode('div.thumbArtwork', 5) || target.findParentNode('div.itemWithCover', 5)))
+		else if (target.dom.localName = 'img' && !target.findParentNode('span.browsedbControls', 3) && !target.findParentNode('div.playlistControls', 3) 
+				&& (target.findParentNode('div.thumbArtwork', 5) || target.findParentNode('div.itemWithCover', 5)))
 			target = Ext.get(target.findParentNode('div'));
 			
 		var el;