[apaku at gmx.de: Re: KDE/kdevplatform]

[Andreas Pakulat]

On 20.05.09 20:52:58, David Nolden wrote:
> Am Mittwoch 20 Mai 2009 18:09:22 schrieb Andreas Pakulat:
> > On 20.05.09 17:26:33, David Nolden wrote:
> > > Am Mittwoch 20 Mai 2009 17:13:53 schrieb Andreas Pakulat:
> > > > The API however should be on Sublime::Area as far as I understood, so
> > > > plugins would need to do
> > > >
> > > > uiController()->activeMainWindow()->area()->findToolView()
> > > >
> > > > Oh and why are we supplying raise-support? After all thats already
> > > > available by calling View::requestRaise(); So in fact I think we should
> > > > have findToolView( bool create = false ) on Sublime::Area and plugins
> > > > can then do
> > > >
> > > > ...findToolView( true )->requestRaise();
> > > >
> > > > Or just get the toolview if they want just that.
> > > >
> > > > Andreas
> > >
> > > But I read comments in the headers that said "Linking to sublime is
> > > forbidden", or was that comment deprecated?
> >
> > Where? we need to remove them, but I can't see anything in sublime sources.
> > Linking to shell is forbidden, but sublime is just fine - IMHO.
> >
> The last comment at the bottom in iuicontroller.h.

Fixed.
 
> Hmm well, then maybe it would be better to change activeMainWindow to return a 
> Sublime::MainWindow, and then putting the API in there.

As I said, IMHO this belongs to the Area class, because that is really
the one that "makes up" the content of a mainwindow (i.e. toolviews,
their positions and opened views).

Andreas

-- 
You're growing out of some of your problems, but there are others that
you're growing into.