Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 421462 - [Model] Enable representation of dialogs and wizards in the E4 model
Summary: [Model] Enable representation of dialogs and wizards in the E4 model
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Eric Moffatt CLA
QA Contact: Paul Elder CLA
URL:
Whiteboard:
Keywords:
: 324956 (view as bug list)
Depends on:
Blocks: 432311
  Show dependency tree
 
Reported: 2013-11-11 10:48 EST by Paul Elder CLA
Modified: 2014-08-12 11:12 EDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Elder CLA 2013-11-11 10:48:47 EST
The E4 model currently has do representation for dialogs and wizards. Add MDialog and MWizard types (extending MWindow) to enable their representation.
Comment 1 Paul Elder CLA 2013-11-13 10:47:35 EST
Initial round of changes pushed to Gerrit for review:

https://git.eclipse.org/r/18343

We will get these committed and into next week's I build so that people can start experimenting with implementations.
Comment 2 Paul Elder CLA 2013-11-14 08:56:42 EST
Pushed to master as:

https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=2b33bfa2e241ec0647a2d3bbd2e1c0070f970a1b

Please provide comments on your progress and problems with implementing renderers for MDialog/MWizard.
Comment 3 Paul Webster CLA 2013-11-14 13:24:24 EST
(In reply to Paul Elder from comment #2)
> Pushed to master as:
> 
> https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/
> ?id=2b33bfa2e241ec0647a2d3bbd2e1c0070f970a1b

There are 4 errors in API tools:

Description	Resource	Path	Location	Type
The major version should be incremented in version 1.1.0, since API breakage occurred since version 1.0.0	MANIFEST.MF	/org.eclipse.e4.ui.model.workbench/META-INF	line 5	Version Numbering Problem
Missing @since tag on createWizardDialog()	MBasicFactory.java	/org.eclipse.e4.ui.model.workbench/src/org/eclipse/e4/ui/model/application/ui/basic	line 121	@since tag problem
The method org.eclipse.e4.ui.model.application.MApplication.getDialogs() in an interface that is intended to be implemented has been added	MApplication.java	/org.eclipse.e4.ui.model.workbench/src/org/eclipse/e4/ui/model/application	line 138	Compatibility Problem
Missing @since tag on createDialog()	MBasicFactory.java	/org.eclipse.e4.ui.model.workbench/src/org/eclipse/e4/ui/model/application/ui/basic	line 112	@since tag problem

See my proposed fix in  https://git.eclipse.org/r/18399

Your API baseline should be set to 4.3.0


PW
Comment 4 Paul Elder CLA 2013-11-14 14:58:10 EST
Pushed a revised patch that put the @noimplement tag into the .ecore model (for all types):

https://git.eclipse.org/r/18399
Comment 5 Paul Elder CLA 2013-11-18 13:01:50 EST
Pushed updated UIEvents to reflect the new model fields.

https://git.eclipse.org/r/18516
Comment 6 Paul Elder CLA 2013-11-18 15:36:51 EST
(In reply to Paul Elder from comment #5)
> Pushed updated UIEvents to reflect the new model fields.
> 
> https://git.eclipse.org/r/18516

Published to master as:

https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=fa91fd45ca025787f1bf6e04e7e042ad7424c2ec
Comment 7 Wim Jongman CLA 2014-01-16 09:43:27 EST
I am trying to implement these in the model editor but they don't extend MContribution and hence I cannot set the implementing class. I sort of expected that.. 

Is the idea behind the implementation of Dialogs to behave just like windows, i.e. as containers for elements like MPart? Interesting. Can you try to explain how you visualize the MDialog implementation?
Comment 8 Lars Vogel CLA 2014-04-08 11:23:05 EDT
Isn't that already implemented?
Comment 9 Lars Vogel CLA 2014-04-28 16:21:49 EDT
I think Paul Elder fixed that. Marking as fixed.
Comment 10 Eric Moffatt CLA 2014-05-01 10:04:12 EDT
Verified in 4.4.0.I20140501-0200.

Still no idea what to do with these though....
Comment 11 Wim Jongman CLA 2014-05-02 16:50:51 EDT
(In reply to Lars Vogel from comment #8)
> Isn't that already implemented?

What do you mean Lars?
Comment 12 Lars Vogel CLA 2014-08-12 11:05:20 EDT
(In reply to Wim Jongman from comment #11)
> What do you mean Lars? 
This bug was still open, so I marked it as fixed.
Comment 13 Lars Vogel CLA 2014-08-12 11:05:28 EDT
*** Bug 324956 has been marked as a duplicate of this bug. ***