Bug 15030 - Make Cover art resizing introduced in 7.4.1 configurable
: Make Cover art resizing introduced in 7.4.1 configurable
Status: RESOLVED FIXED
Product: SB Radio
Classification: Unclassified
Component: UI Skin
: Include FW version in comment
: PC Windows Vista
: -- enhancement with 8 votes (vote)
: Future
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-11-02 23:43 UTC by Stefan Hansel
Modified: 2009-11-17 11:41 UTC (History)
4 users (show)

See Also:
Category: Feature


Attachments
screenshots of two available NP views in 7.5 firmware (64.05 KB, image/png)
2009-11-17 11:41 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Hansel 2009-11-02 23:43:00 UTC
Your current cover art resizing introduced in 7.4.1 in the Now-Playing screen crops the edges which looks ugly in most cases.

As an example take your own screenshot from the release notes (http://wiki.slimdevices.com/index.php/Image:Squeezeplay0002.jpg) you can't even read the upper part because:
1) it is cropped and
2) it is overlayed

Here are some further cover arts which render very annoying:
http://www.musicline.de/cover/Mercury,Freddie_Freddie+Mercury+Album_77778099925.jpg
http://www.emiextranet.com/Packshots/Med/0724385031625_M.jpg
http://fourmusic.allmybands.de/img/band/diskografie/clueso_e9de_xxl.gif
http://projectlore.com/wp-content/uploads/2008/10/galore.jpg
http://www.lapsus-gil.de/lover/lov37176.jpg

All those musicians don't deserve to get theirs heads cut-off !

In the forums some users also started to complain: http://forums.slimdevices.com/showthread.php?t=70776

Please revert the old resizing or make it at least configurable.
Comment 1 Digital Mitch 2009-11-03 03:29:08 UTC
This thread has some good mock-ups:

http://forums.slimdevices.com/showthread.php?t=69867&highlight=cover+size&page=4

Another thread (or Bug), which I can't find also discussed mechanisms for changing the now playing screen, which include:-
1) "Now Playing" and/or "Size" buttons on old IR  (former changing style, the latter size within style?)
2) rotating the radio/SBC wheel whilst in Now Playing mode (currently does nothing)

I think configurability is the key as there are different biases towards albumart, text or time.
Comment 2 James Richardson 2009-11-03 07:45:15 UTC
This would make the start of a good Community Applet.  Maybe you could ask
someone on our forums to create a new applet with these enhancements in them.
Comment 3 Digital Mitch 2009-11-03 11:11:31 UTC
I think that route is blocked by the license restrictions on the current code

https://bugs-archive.lyrion.org/show_bug.cgi?id=14194

not sure how to add that as a block.
Comment 4 Stefan Hansel 2009-11-03 11:41:34 UTC
Copying the whole Now-Playing applet to just have the artwork resizable is a bit heavyweight I think.
And it is one of the most complicated applets in squeezeplay, with many bugs subtle bugs solved when reading the sourcecode.

Would you also accept patches to the original applet to make it configurable ?
Wouldn't make sense though if you have any guideline to reduce settings or so.

And if there isn't a chance that it could get integrated into the official line of code I wouldn't dare to put work into into it as this patch had to be manually employed after each firmware upgrade.
Comment 5 James Richardson 2009-11-03 12:20:22 UTC
Patches are always welcome :)
Comment 6 Ben Klaas 2009-11-17 11:39:10 UTC
In 7.5, I've added a second "view" for the NP screen. When you come into the NP screen, turn the knob a half turn and you will get the smaller artwork. Another half turn and the full bleed effect comes back.

Screenshots coming...
Comment 7 Ben Klaas 2009-11-17 11:41:28 UTC
Created attachment 6320 [details]
screenshots of two available NP views in 7.5 firmware