Bug 6826 – EN & Classic skins - Artist > All Songs cover isn't clickable

Bug 6826 - EN & Classic skins - Artist > All Songs cover isn't clickable

Summary:

EN & Classic skins - Artist > All Songs cover isn't clickable

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Skins
Version:	7.0
Platform:	PC Windows XP

Importance:	P2 enhancement (vote)
Target Milestone:	7.x
Assigned To:	Michael Herger

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-01-28 08:32 UTC by Jim McAtee
Modified:	2009-07-31 10:16 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
cursor and link (2.27 KB, patch) 2008-01-28 09:44 UTC, KDF	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jim McAtee 2008-01-28 08:32:07 UTC

When in the artwork view of the EN or Classic skin there's no link on the cover to the tracklist for All Songs.  Also, a minor point, but ideally the cursor should change to a pointer when hovering over the album covers.

Comment 1 KDF 2008-01-28 09:33:56 UTC

Artwork is for the album popup.  This is intended for albums only (if ($('albumPopup') && attributes.match('album.id')) {).  See change 13234. 

I would, however be more clear if the code were completely removed around the link instead of just blocking the javascript functionality.

Comment 2 KDF 2008-01-28 09:44:18 UTC

Created attachment 2745 [details]
cursor and link

This patch sets the cursor type for artworkImage style, bypasses the javascript for the album popup and turns the allsongs into a regular link instead.

Comment 3 Michael Herger 2008-02-14 01:41:52 UTC

change 17536 - thanks Kevin!

Comment 4 James Richardson 2008-05-07 15:10:21 UTC

Verified fixed in 7.0.1 - 19325

Comment 5 James Richardson 2008-05-15 12:27:27 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

Comment 6 Chris Owens 2009-07-31 10:16:30 UTC

Reduce number of active targets for SC