Configurable layout in Kopete

Roman Jarosz kedgedev at gmail.com
Tue Feb 24 18:40:08 CET 2009


On Tue, 24 Feb 2009 09:50:19 +0100, Nikolaj Hald Nielsen  
<nhnfreespirit at gmail.com> wrote:

>> Hi Roman,
>> Good to know that you're using our code. I can't see any reason to not
>> make the above changes where relevant and appropriate. However, if
>> you're already working with it, then it would be best if you could
>> simply send a patch which addresses these issues (as you've probably
>> already made these changes locally).
>
> I agree with Seb. All of your suggestions seem reasonable. Just send
> us a patch, and if it does not break anything, I will commit it.
>
> - Nikolaj

Here's the patch, the only thing which has left, is to make
"application/x-amarok-tag-token" runtime configurable
(will make second patch later).

Is it ok to commit?

Roman
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarokfix1.diff
Type: text/x-patch
Size: 13821 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090224/99029c81/attachment.diff 


More information about the Amarok-devel mailing list