Page 1 of 2

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 11:44 am
by anonbeat
Just commited a change that allow from the library to embed the album cover to the album tracks.
It works with multiple albums selected and every album will get its cover. If the album have no cover it will remove any cover the files may have embedded.

Thanks for your help

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 12:30 pm
by camaron
anonbeat wrote:Just commited a change that allow from the library to embed the album cover to the album tracks.
It works with multiple albums selected and every album will get its cover. If the album have no cover it will remove any cover the files may have embedded.

Thanks for your help
Thanks anonbeat, this is a very neat featue.
Here is a consideration:
In the process to get/find covers for my albums I never worried about the size of the bitmaps. Some are into the few Mbs. Since I started (recently) embedding the covers into the files It crossed my mind that surely there is a reasonable limit as to how big the files can be?

Do you (or anyone) know what this limit is? If there is a limit do you think Guayadeque should some how be aware of it?

Just thinking that if it is not reasonable to embed a 2 MB jpg into a 3 MB mp3 files then the practicality of this new feature is in great measure lost as I would have to be navigating with Nautilus to the folders and then possibly editing the covers manually to reduce the size.

Guayadeque transcodes on the fly now. Could it resize jps on the fly too?

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:07 pm
by anonbeat
Yes of course It could be done, in fact, it does that for audio player devices.

Its needed to define somewhere the max size for an embedded image.

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:18 pm
by camaron
Thank anon,
Do you plan to generalize this feature soon?
Just thinking that I can indeed wait for something like that before I get into all the hassle of having to edit thousands of jps manually...

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:27 pm
by anonbeat
I think for now I can enable a limit in embedded images of 600x600

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:36 pm
by camaron
anonbeat wrote:I think for now I can enable a limit in embedded images of 600x600
Happy coincidence: That's in fact the size I've been using manually :)

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:43 pm
by anonbeat
Changed already in svn

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:53 pm
by camaron
anonbeat wrote:Changed already in svn
Bloody hell....

Added option Embed Cover in Album Tracks

Posted: Sat Oct 23, 2010 1:57 pm
by camaron
Just for the record: Not even 1 1/2 hours between feature request and implementation...

Added option Embed Cover in Album Tracks

Posted: Sun Oct 24, 2010 11:04 pm
by mekanix
Hi anon and all!

I really like this!!! For me it's really useful!!!!

I think it could be a bit better it there will be an option to automatic add cover to files when a cover is selected for the album!