[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


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


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