[PATCH] minor sublime modifications

[Andreas Pakulat]

On 29.07.08 09:44:39, Manuel Breugelmans wrote:
> On Tuesday 29 July 2008 02:43:17 Matt Rogers wrote:
> > 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?
> 
> Joe starts kdevelop4. Joe requests 3 different code coverage reports, which get 
> initiated through a context menu action. kdevelop4 adds these 3 toolviews with 
> IUiController->addToolView. Joe opens the 'add new tool view' box and finds 3 
> 'Coverage Report' entries there. Joe files a bugreport.
> 
> Problem being that addToolView does 2 different things:
> 1. register the factory
> 2. spawns the toolview (if areas are restored yet)
> 
> And yes, one could implement it with tabs/dropdownboxes in a single toolview 
> instead of these throw-away views.

Ah, so I didn't imagine things :) And now I start to understand things a
bit better. 

Ok, this is broken by design (both sublime and your way of using
toolviews) and I'm sorry for not noticing earlier. I don't have time
right now to think about how to fix that as I do understand its not
quite that easy to fix. 

One way of course is doing like outputview, though it still uses tabs it
will use a plain small selection-list on the left-side sooner or later.

Andreas

-- 
You are capable of planning your future.