Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [e4-dev] More Model Question and Proposals


I think you're correct that MPerspective should not be MPSCElement, its only correct location is directly under some MPerspectiveStack (which should be restricted to *only* MPerspectives)..

The reason that MWindow is an MPSCElement is that we have to support 'detached' windows. This means that MWindow has to be able to be contained in other MWindow's children lists (in the legacy code they're actually children of some MPerspective).

We are trying to support two different styles; basic e4 and legacy eclipse and I think that some of the confusion comes from there. It reasonable (I think) that an RCP app may want to define 'satellite' (floating) windows otherwise I may have defined MWindow as only being 'legal' under MPerspective.

Tom, which of our elements do you see as being MWindowElement rather than MPSCElement ? We should see if we can define what each separate element *should* contain, once we know that then setting up the containment in the model will follow naturally. One way to look at this is to ask "If i right-click on an element in the model editor which elements should be in the 'Add' menu?".

Eric



From: Remy Suen/Ottawa/IBM@IBMCA
To: E4 Project developer mailing list <e4-dev@xxxxxxxxxxx>
Date: 03/11/2010 07:58 AM
Subject: Re: [e4-dev] More Model Question and Proposals
Sent by: e4-dev-bounces@xxxxxxxxxxx





On Wed, Mar 10, 2010 at 5:44 PM, Tom Schindl <tom.schindl@xxxxxxxxxxxxxxx> wrote:
> I think MPerspective and MWindow have NO reason to be a MPSCElement.
>
> I'm not sure about MPerspectiveStack yet but probably a good thing
> would be to introduce a MWindowElement-Interface similar to MPSCElement.
>
> This would give us:
>
> ---------8<---------
> + UElement
> + MWindowElement
> + MPerspectiveStack
> + MPSCElement & MWindowElement
> + MPart
> + MPartSashContainer
> + MPartStack
> ---------8<---------

MWindow is currently an MElementContainer<MPSCElement>. This suggested change would mean an MWindow is an MElementContainer<MUIElement> I guess?

Regards,
Remy

----------
Remy Suen
Eclipse Platform/UI Committer
IBM Ottawa
613-356-5162
_______________________________________________
e4-dev mailing list
e4-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/e4-dev



Back to the top