[PATCH] minor sublime modifications

Andreas Pakulat apaku at gmx.de
Mon Jul 28 12:13:03 UTC 2008


On 28.07.08 12:46:20, Manuel Breugelmans wrote:
> Hi,
> 
> Attached a patch which does 3 things:
> 
> */ allow to add toolviews without registering them in the 'add new toolview' 
> box

Why?

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

See my other mail, there's no way to exclude storing the position and
quite frankly I don't see why we shouldn't be storing the "visibility"
state. If the resultsview is empty, so be it. It shouldn't be shown in
development area anyway, there should be a dedicated "Test" area, which
can always show resultsview and testrunner.

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

That one sounds good - without looking at the patch. I'll do that later
today.

Andreas

-- 
Your best consolation is the hope that the things you failed to get weren't
really worth having.




More information about the KDevelop-devel mailing list