[Panel-devel] containment switching patch

[Jeremy]

On Monday 26 November 2007 14:32:33 Jeremy wrote:
> On Monday 26 November 2007 14:15:55 Aaron J. Seigo wrote:
> > On Monday 26 November 2007, Jeremy wrote:
> > > On Monday 26 November 2007 03:12:43 Aaron J. Seigo wrote:
> > > > On Sunday 25 November 2007, Jeremy wrote:
> > > > > I have a patch here that works mostly.  Any input is appreciated.
> > > > > The idea is to reparent an applet when it is moved into another
> > > > > containment (just
> > > >
> > > > i think that the addChild call should be moved to
> > > > Containment::addApplet. this prevents need to call setParentItem, and
> > > > it also ensures that the applet gets associated with the correct
> > > > layout.
> > >
> > > how could we do that without breaking Widget::Widget on line 101?
> >
> > what would break? all it does it move the addChild call out of the full
> > Containment::addApplet method that loads it from disk and into the new
> > Containment::addApplet(Applet* applet) method. it's just rearanging some
> > code to make sure that addChild is always called as it must be.
>
> ok, got it, I misunderstood.

Checked in, but now when I call addChild from within addApplet() it moves even 
the applet... :(  Also when in zoomed out view, the whole containment moves 
just like when you drop a new applet onto it :(  is Widget::updateGeometry() 
that's called at the end of addChild causing problems?

Jeremy

>
> > > > > desktop to desktop containment so far, no tricky panel stuff yet).
> > > > > Anyway,
> > > >
> > > > =)
> > > >
> > > > > then you can test while zoomed out.  The only thing missing is when
> > > > > you drop the applet on the new containment, the handle jumps to a
> > > > > differentt location.
> > > >
> > > > we can tackle that next i suppose ...
> > > >
> > > > otherwise, looks good..
> > >
> > > Should I commit today as it's a bic change (added some methods to the
> > > api)?
> >
> > sure.
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel at kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel