[prev in list] [next in list] [prev in thread] [next in thread]
List: kdevelop-devel
Subject: Re: Area model
From: Vladimir Prus <ghost () cs ! msu ! su>
Date: 2008-05-11 13:34:16
Message-ID: 200805111734.17462.ghost () cs ! msu ! su
[Download RAW message or body]
On Sunday 11 May 2008 16:30:10 Andreas Pakulat wrote:
> On 05.05.08 23:20:54, Vladimir Prus wrote:
> > On Monday 05 May 2008 22:50:41 Alexander Dymo wrote:
> > > Vladimir, thanks for the explanation (in this and your other mail). At least
> > > now I understand much better what areas are and how we want to use them. I
> > > didn't know that before I started implementing areas.
> > >
> > > The current implementation still bothers me a bit. We have two separate
> > > concepts - area types and areas which are implemented using the same Area
> > > class and distinguished by uicontroller as "default" area shown nowhere
> > > or "clone of the default" shown in the mainwindow. I'd like to try separating
> > > these two in the code as well into Area and AreaType classes. AreaType would
> > > be the default area added in the code or by plugins and Area class would be
> > > the same as it was before - just collection of views shown in the mainwindow.
> >
> > How those two would be related? The joy of the current model is that visible
> > areas are indeed just clones of the default areas, and resetting areas to
> > default is a simple clone. In smart words, this is called "Prototype" pattern.
> > I'm not sure using two separate classes would keep this simplicity.
> >
> > What personally bothers me right now is that we have this default/clone model,
> > but outside of it, you can set any random area to a window, thereby probably
> > breaking lots of things. I'd suggest making that impossible :-)
>
> BTW: Please update the points in our wiki when you've figured out how
> Areas and Sublime should work.
Wiki? I'd think this should be a comment in the code, most likely for the
Sublime::Controller class.
- Volodya
_______________________________________________
KDevelop-devel mailing list
KDevelop-devel@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic