Menu branch

Eike Hein hein at kde.org
Sat Apr 20 07:02:37 UTC 2013


On Friday 19 April 2013 11:32:48 Ignat Semenov wrote:
> setLayout() and setAlignment() both trigger a complete relayout of the view.
> Relayouting the view twice is suboptimal, however, in the code there is a
> 10ms timer to delay the relayout so as not to block the caller. Should that
> timer be enough to ensure that when those two funcitons are called one
> after another, the view is only relayouted once? Maybe increase the delay a
> bit?

The timeout can't be delivered until the code flow returns to the event
loop anyway.

I haven't had time to read the diffs yet though, I'll take a look once I
work through some other todos.


> Best regards,
> Ignat Semenov

Cheers,
Eike


More information about the Plasma-devel mailing list