[PATCH] minor sublime modifications

Andreas Pakulat apaku at gmx.de
Tue Jul 29 06:27:59 UTC 2008


On 28.07.08 19:43:17, Matt Rogers wrote:
> On Jul 28, 2008, at 5:46 AM, Manuel Breugelmans wrote:
> > */ move invocations of factory->viewCreated() from UiController to
> > Sublime::View [there were some places this call was missing, like on  
> > area
> > restoration from config]. I think this is the right place to invoke  
> > it anyway.
> >
> > Objections, alterations?
> >
> > Btw I think the current way of storing area configuration has a  
> > flaw: no support
> > for multiple identical ID'ed toolviews in a single area, eg 2  
> > document views.
> 
> Toolview ids should be unique per instance of that toolview (i.e. 2  
> document views, two different ids)

You're getting it wrong :) "id" is the id of the toolview factory, not
the toolview itself. We currently only support storing toolview factory
information, not actual toolviews. I agree thats a problem.

Andreas

-- 
Be cheerful while you are alive.
		-- Phathotep, 24th Century B.C.




More information about the KDevelop-devel mailing list