Bug 4647 – Nokia770 song info page doesn't use server-resized artwork

Bug 4647 - Nokia770 song info page doesn't use server-resized artwork

Summary:

Nokia770 song info page doesn't use server-resized artwork

Status:	CLOSED FIXED

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

Importance:	P2 enhancement (vote)
Target Milestone:	---
Assigned To:	Chris Owens

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2007-01-09 03:40 UTC by Nigel Birch
Modified:	2008-12-18 11:12 UTC (History)
CC List:	2 users (show)

See Also:
Category:	---

Attachments
Changes Noka770 song info page to use server-resized artwork (723 bytes, patch) 2007-01-09 03:41 UTC, Nigel Birch	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 Nigel Birch 2007-01-09 03:40:27 UTC

On the song info page (HTML/Nokia770/songinfo.html), artwork doesn't use server-based resizing (uses browser-based resizing).  The attached patch against 6.5.1 makes it use server-resized artwork.

The change resizes to a height of 220 px (since the HTML tag specifies just a height tag).

Thanks.

Comment 1 Nigel Birch 2007-01-09 03:41:05 UTC

Created attachment 1764 [details]
Changes Noka770 song info page to use server-resized artwork

Comment 2 KDF 2007-05-17 17:04:13 UTC

ben, input on this?  seems like a reasonable idea given the geometry limits of the nokia

Comment 3 Ben Klaas 2007-05-18 10:20:48 UTC

I fixed this in trunk several months ago, but I didn't think it was necessary to work it into the 6.5 branch (still don't).

I'm moving this to fixed with target of 7.0. Feel free to reopen if you feel I'm in error.

Comment 4 Chris Owens 2008-03-07 09:04:01 UTC

This bug is being closed since it was resolved for a version which is now released!  Please download the new version of SqueezeCenter (formerly SlimServer) at http://www.slimdevices.com/su_downloads.html

If you are still seeing this bug, please re-open it and we will consider it for a future release.