On Sunday 22 February 2009 15:10:47 Roman Jarosz wrote: > On Sun, 22 Feb 2009 21:02:33 +0100, Matt Rogers <ma...@kde.org> wrote: > > On Sunday 22 February 2009 05:26:01 Roman Jarosz wrote: > >> Hi, > >> > >> here's little preview of contact list layout for model/view branch, most > >> code is taken > >> from Amarok. The patch is almost ready I just need to polish the > >> configuration dialog > >> (emit changed signal and add create/delete layout buttons) for this I > >> want > >> to wait for > >> improved configuration dialog which should be merged into Amarok this > >> week. > >> > >> The classes in kopete/config/appearance/layout directory are copied from > >> Amarok > >> and weren't changed. > >> > >> Screenshot: http://kedge.wz.cz/kopete/mvlayout3.png > >> > >> Any suggestions welcome. > >> > >> Regards, > >> Roman > >> > >> PS. I had to put contactlist classes into library so I can use it in > >> configuration. > > > > Implementing the missing functionality from the current contactlist > > would be > > more useful IMHO. What's the point of adding customization when it > > doesn't even > > work right? > > Actually this is implementing missing functionality because we lost > status message and protocol icons, so this patch implements them :P > and what's the point of making 3 hard-coded layouts when we can make it > configurable without much work (taking Amarok code)? > > > Matt - (who's tired of people making things look pretty to gloss over > > the missing > > functionality) > > > > _______________________________________________ > > kopete-devel mailing list > > kopete-devel@kde.org > > https://mail.kde.org/mailman/listinfo/kopete-devel > > Roman
Well, my original intention was to support adium contact list styles but i don't think i'll ever get around to that. :( Can you post the patch to reviewboard? Reviewing things on mailing lists is so 1990s. :) -- Matt _______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel