Bug 176882

Summary:	Giant cover image in Current Track applet
Product:	[Applications] amarok	Reporter:	Matthias Weiss <matias.liberta>
Component:	Context View	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	normal	CC:	lfranchi, marcusdelyon, shlomif, vianasw
Priority:	HI
Version First Reported In:	2.0-SVN
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Screenshot crop with big album image Screenshot crop with small album image

Description Matthias Weiss 2008-12-04 12:37:32 UTC

Version:           2.0-rc (using KDE 4.1.3)
OS:                Linux
Installed from:    Debian testing/unstable Packages

In my mp3 filetree I have for every music album an album folder, in every album folder there is a cover image with the filename "cover.jpg".

After starting from a fresh amarok 2.0 rc 1 installation and adding mp3 folders to the collection I see a cover image of approx. 270 x 270 pixels in the "Current Track Info View" when playing a song.

After quiting amarok and restarting again the cover images in the "Current Track Info View" have reduced the size to approx. 150 x 150 pixels.

See the 2 attachements.

Comment 1 Matthias Weiss 2008-12-04 12:38:26 UTC

Created attachment 29033 [details]
Screenshot crop with big album image

Comment 2 Matthias Weiss 2008-12-04 12:39:07 UTC

Created attachment 29034 [details]
Screenshot crop with small album image

Comment 3 Matthias Weiss 2008-12-04 12:39:57 UTC

Oh, and I forgot the obvious: I'd like that the album images stay big.

Comment 4 Mikko C. 2008-12-04 12:42:06 UTC

FYI, the small image is how it's supposed to look like in Amarok 2.0.
The initial one is a bug.

Comment 5 Matthias Weiss 2008-12-04 12:47:57 UTC

(In reply to comment #4)
> FYI, the small image is how it's supposed to look like in Amarok 2.0.
> The initial one is a bug.
> 

:-((

No chance that the big one returns? ;-))))

Comment 6 Lydia Pintscher 2008-12-04 23:00:00 UTC

Hmmm I thought we fixed this for RC 1...

Comment 7 Mikko C. 2008-12-21 13:42:57 UTC

*** Bug 178193 has been marked as a duplicate of this bug. ***

Comment 8 Seb Ruiz 2008-12-27 11:44:47 UTC

*** Bug 177447 has been marked as a duplicate of this bug. ***

Comment 9 Mark Kretschmann 2008-12-27 11:55:06 UTC

I could now reproduce the problem. Here's a screenshot:

http://mark.kollide.net/amarok2_giant_cover.png


How to reproduce:

* Open shell
* export KDEHOME=$HOME/kdetestconfig
* Start amarok from shell

Comment 10 Matthias Weiss 2008-12-30 15:40:58 UTC

(In reply to comment #9)
> I could now reproduce the problem. Here's a screenshot:
> 
> http://mark.kollide.net/amarok2_giant_cover.png
> 
> 
> How to reproduce:
> 
> * Open shell
> * export KDEHOME=$HOME/kdetestconfig
> * Start amarok from shell
> 

Yes, also in Amarok 2.0 there is still the same bug.

Again, I'd like to stress that I like the large cover image better.

On the other hand Leo Franchi wrote in his blog today that the context view will be reworked ..., so dunno whether it's worth to look further into this bug.

Comment 11 Leo Franchi 2008-12-31 07:47:09 UTC

the CV lays out applets on startup, and it needs to guesstimate what size to put them at. the problem is that this happens  before the mainwindow has its final size, so when you ask the window you get erroneous results.

william's fix for this was to write to config the size of each applet on exit, and restore to that on startup if possible. so, with fresh config this is not going to work.

i don't know how much blood/sweat/tears i want to spend on the 2.0 CV as i'm busy making the 2.1 CV that much cooler. but it definitely needs some love.

Comment 12 Lydia Pintscher 2009-03-14 12:18:24 UTC

I havn't seen this happen in a long time. Marking as fixed. Please reopen if it still happens with current SVN.