[Fwd: Re: [Bug 164805] Multiple columns in kick off]

Luiz Felipe Talvik talvik at gmail.com
Tue Apr 28 20:21:34 CEST 2009


What does it make it "not complete":
-the items paint text outside the defined area
-keyboard navigation left/right
-multiple columns on the apps tab
    I don't understand one line of code that does this tab
-columns threshold based on the font size(currently fixed width)

For those that discussed the previous patch(Aaron), I changed the
behavior of the layout to something similar to gridlayout per section.
The previous was section per column, which became extremely complex.
(Before I could complete the patch I had some problems and them forgot
about it)

Why kickoff implements all the layout spacing, margins, placing
items...? Why doesn't it use something like VBoxLayout? Is there are
reason I can't change to a grid layout?
d, q objects and a private class is present in almost all classes from
Robert Knight, is this a common practice where can I read about it?


On Tue, Apr 28, 2009 at 2:05 PM, Marco Martin <notmart at gmail.com> wrote:
> On Tuesday 28 April 2009, Bernhard Friedreich wrote:
>> Hey!
>>
>> I'm forwarding this mail because I'd really like to see multiple columns
>> in kickoff (in 4.3). Afaik hard feature freaze is next week so it must
>> go in "now"...
>>
>> The original author of the patch has no time to complete it and I don't
>> know if I'll have the skills/time to do so.. so I just bring up this
>> topic again in hope that someone can volunteer and bring that patch to
>> shape (if even neccessary).
> what does it make it "not complete"?
>>
>> I applied it some time ago and it worked without any major problems. (I
>> would have tried with newer trunk but unfurtunately I'm atm bound to
>> Windows cause of school :-( )
>>
>> Attached you can find the patch from Luiz Felipe Talvik. All the credit
>> goes to him.. I was just the one reminding him of this feature from time
>> to time :D
>>
>> I really hope this will end up in trunk.. otherwise I'll try to get it
>> ready at least for 4.4..
>>
>> Thanks!
>>
>> Best regards,
>> Bernhard Friedreich
>>
>> PS: If anyone wants to put this on reviewboard.. no problem.. I just
>> couldn't figure out what to do with the base path thingy ^^
>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>


More information about the Plasma-devel mailing list