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

Bug 312245

Summary: Patch to improving "org.eclipse.gmt.modisco.infra.browser" extensibility
Product: [Modeling] MoDisco Reporter: Nicolas Payneau <npayneau>
Component: InfrastructureAssignee: Nicolas Bros <nicolas.bros>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P4 CC: gdupe, hugo.bruneliere, modisco.web-inbox, nicolas.bros
Version: 0.9.0Flags: nicolas.bros: indigo+
Target Milestone: 0.9.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch to improve "org.eclipse.gmt.modisco.infra.browser" extensibility gdupe: iplog+

Description Nicolas Payneau CLA 2010-05-10 08:40:46 EDT
Build Identifier:  20100318-1801

Through the development of tools for not expert users in MDA, I chose to use MoDisco browser. So I need to hide some browser elements allowing to have simplified ui.
To satisfy my needs, I had refactoring classes “EcoreBrowser.java”, “MetaclassViewer.java” . There is a patch "org.eclipse.gmt.modisco.infra.browser":
     - I added "DND.DROP_COPY|DND.DROP_MOVE" on “DragAdapter.java”
     - I changed the visibility of "createContextMenu" method in "protected" on “MetaclassViewer.java”
     - I changed the method visibility of "createTreeToolBar" in protected on “EcoreBrowser.java”
     - I changed the type of propertySheetPage to IPropertySheetPage (Initially in PropertySheetPage) on “EcoreBrowser.java” 

So, I managed the different type “PropertySheetPage” and “TabbedPropertySheetPage” in the code of “EcoreBrowser.java “. I managed “TabbedPropertySheetPage” to include EEF in MoDisco browser.

Best regards,

Nicolas Payneau


Reproducible: Always
Comment 1 Nicolas Payneau CLA 2010-05-10 08:43:05 EDT
Created attachment 167697 [details]
Patch to improve "org.eclipse.gmt.modisco.infra.browser" extensibility
Comment 2 Nicolas Bros CLA 2010-05-11 08:10:46 EDT
Thank you for your patch. I applied it to the trunk (for 0.9).
Comment 3 Hugo Bruneliere CLA 2010-10-04 12:55:23 EDT
The patch has been successfully applied.