[PATCH] minor sublime modifications

Matt Rogers mattr at kde.org
Tue Jul 29 00:43:17 UTC 2008


On Jul 28, 2008, at 5:46 AM, Manuel Breugelmans wrote:

> Hi,
>
> Attached a patch which does 3 things:
>
> */ allow to add toolviews without registering them in the 'add new  
> toolview'
> box

what's your use case for this?

>
> */ ability to exclude certain toolview ID's from saving to config

again, what's the use case?

>
> */ 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.
>
>
> Manuel


Toolview ids should be unique per instance of that toolview (i.e. 2  
document views, two different ids)
--
Matt





More information about the KDevelop-devel mailing list