Fwd: Re: Workflow Idea for 4.10

2012-03-15 Thread Kevin Ottens
Hello, There's an ongoing discussion on the plasma-devel list about workflows and especially git. Since that's a topic we touched during Platform11, and that my thinking there evolved quite a bit since I'm forwarding my contribution to the thread here as well, at least for reference purpose. I

Re: KIcon API replacement (Re: patches on kcolors)

2012-03-15 Thread Christoph Feck
On Thursday 15 March 2012 23:29:49 Kevin Ottens wrote: > On Thursday 15 March 2012 21:21:07 David Faure wrote: > > On Thursday 15 March 2012 19:53:36 Kevin Ottens wrote: > > > Client code would then look like: > > > QIcon i = KDE::Icon("foo"); > > > instead of: > > > QIcon i = KDE::loadIcon("foo");

Re: KIcon API replacement (Re: patches on kcolors)

2012-03-15 Thread Kevin Ottens
On Thursday 15 March 2012 21:21:07 David Faure wrote: > On Thursday 15 March 2012 19:53:36 Kevin Ottens wrote: > > Looks good to me. We probably want also an overload without the iconLoader > > parameter I guess. > > It default to 0, so I assume you mean for the case where someone wants to > specif

Re: KF5 Volunteer day 1: Done

2012-03-15 Thread David Faure
On Thursday 15 March 2012 23:18:19 Kevin Ottens wrote: > As for not having "cleaned up and commented test app"... We definitely have > those at least in solid. I made crude manual test applications back in the > days, but we also have nice examples splitted in several steps to be > followed as tut

Re: KConfig and KGlobalSettings (user settings for widgets)

2012-03-15 Thread Kevin Ottens
On Tuesday 13 March 2012 22:47:06 David Faure wrote: > On Tuesday 13 March 2012 22:04:18 Alexander Neundorf wrote: > > There it is quite similar: do we just provide independent libraries, where > > every developer can/must take care that the libraries follows KDE > > conventions how it is built an

Re: KF5 Volunteer day 1: Done

2012-03-15 Thread Kevin Ottens
Hello, On Sunday 11 March 2012 09:14:41 David Faure wrote: > On Sunday 11 March 2012 08:29:53 David Faure wrote: > > On Monday 27 February 2012 22:02:53 Kevin Ottens wrote: > > >- Manual tests in: staging/kguiaddons/tests > > > > You mean staging/kguiaddons/examples, I think. > > Oh, you're ri

Re: KIcon API replacement (Re: patches on kcolors)

2012-03-15 Thread David Faure
On Thursday 15 March 2012 19:53:36 Kevin Ottens wrote: > On Saturday 10 March 2012 13:03:10 David Faure wrote: > > On Saturday 10 March 2012 11:49:21 David Faure wrote: > > > Maybe we want to make it a method of > > > KIconLoader/KIconEngine instead? > > > > KIconLoader::loadIcon is already taken

Re: KIcon API replacement (Re: patches on kcolors)

2012-03-15 Thread Kevin Ottens
On Saturday 10 March 2012 13:03:10 David Faure wrote: > On Saturday 10 March 2012 11:49:21 David Faure wrote: > > Maybe we want to make it a method of > > KIconLoader/KIconEngine instead? > > KIconLoader::loadIcon is already taken (and returns a QPixmap), and > KIconEngine is an internal class. > >