Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 317417

Summary: [Model] StackElement should be abstract and interface
Product: [Eclipse Project] e4 Reporter: Thomas Schindl <tom.schindl>
Component: UIAssignee: Thomas Schindl <tom.schindl>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: bokowski, john.arthorne, pwebster, remy.suen
Version: 0.9   
Target Milestone: 1.0 RC1   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
patch none

Description Thomas Schindl CLA 2010-06-21 07:16:41 EDT
StackElement is only a marker for improved containment hierarchies. We should at least make it abstract but also as an interface like WindowElement, ...
Comment 1 Thomas Schindl CLA 2010-06-21 08:40:41 EDT
Created attachment 172323 [details]
patch

This patch will probably not apply inconjunction with the one from bug #317405
Comment 2 Thomas Schindl CLA 2010-06-22 14:09:30 EDT
anyone a +1?
Comment 3 Eric Moffatt CLA 2010-06-22 15:10:06 EDT
Tom, please no model changes now...

I agree that there have been a number of 'ad hoc' changes to the model as we try to meet the 4.0 SDK release date but I think we should wait until after we get this thing out before making any more changes that are not directly related to this task.

BTW, I'm in complete agreement that we should do a model redux very soon after we go out to pick up some of these defects.
Comment 4 Thomas Schindl CLA 2010-06-23 03:12:55 EDT
This is a none breaking change it only declares an element abstract/interface nobody should create an instance of. I don't see how this could break something.

I'm not trying to bring in none breaking changes but e.g. the reflective editor and Yves generic stuff will allow you to create instance of this.
Comment 5 Thomas Schindl CLA 2010-06-23 10:11:32 EDT
released to HEAD