Re: Panel desktop scripting

2013-11-19 Thread Eike Hein
On Tuesday 19 November 2013 08:38:54 Aaron J. Seigo wrote: > awesome :) > > the same fix is probably also needed in the Plasma2 desktop scripting. > > along with the additions for panel min/max size setting i made the other day > as well. I merged KDE/4.11 to master - if the desktop/shell code

Re: Panel desktop scripting

2013-11-18 Thread Aaron J. Seigo
On Tuesday, November 19, 2013 00:06:11 Eike Hein wrote: > On Monday 18 November 2013 16:49:57 Aaron J. Seigo wrote: > > call flushPendingConstraintsEvents > > > > this kind of use case is exactly why it is a public slot :) > > Thanks, I missed that bit of API :) i carefully hid it ;) > I've add

Re: Panel desktop scripting

2013-11-18 Thread Eike Hein
On Monday 18 November 2013 16:49:57 Aaron J. Seigo wrote: > call flushPendingConstraintsEvents > > this kind of use case is exactly why it is a public slot :) Thanks, I missed that bit of API :) I've added the flush to the implementation of setLocation() in the scripting wrapper, and committed t

Re: Panel desktop scripting

2013-11-18 Thread Aaron J. Seigo
On Saturday, November 16, 2013 20:10:13 Eike Hein wrote: > Thoughts on a clean(er) fix? call flushPendingConstraintsEvents this kind of use case is exactly why it is a public slot :) -- Aaron J. Seigo ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Panel desktop scripting

2013-11-16 Thread Eike Hein
On Saturday 16 November 2013 20:10:13 Eike Hein wrote: > 1. Spinning the event loop in scripting's location >setter so the constraints update is guaranteed to >happen before control is returned to the script. >Dirty ... Just for clarity: Appending 'QCoreApplication::processEvents();' t

Panel desktop scripting

2013-11-16 Thread Eike Hein
Hi, while creating a template for a 'sidebar' panel type, I've noticed that using desktop scripting to set the geometry of a vertical panel doesn't work quite right. Take this following example: var panel = new Panel; panel.location = 'right'; panel.length = 900; panel.height = 200; Given that