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

Bug 472033

Summary: WorkspaceImageDescription as Region or RegionContainer style
Product: [Modeling] Sirius Reporter: Maxime Porhel <maxime.porhel>
Component: DiagramAssignee: Maxime Porhel <maxime.porhel>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P1 CC: florian.barbin, pierre-charles.david
Version: 3.0.0Keywords: triaged
Target Milestone: 3.1.0   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/54409
https://git.eclipse.org/r/54408
https://git.eclipse.org/r/54416
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=272e69bb35772e3a51c6d7554a6cfe7a8d6d0aa9
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=9a545eb3151581530d4285a3738dd9976a76c459
https://git.eclipse.org/r/54565
https://git.eclipse.org/r/54571
https://git.eclipse.org/r/54601
https://git.eclipse.org/r/54660
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=313416a5257555eddb1d62e1ba6412af68b36d28
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=69d9e9b4e4da54c7f62b783036719ce42f3c5056
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=76e801508e7452982cb4efdba9bcd8b37c5cc8d9
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=0b09358624894ac60e5cdcf325a41d44e831f9e0
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=56f29a9696845fdbad9742f15d4c5fdf57fdebb5
Whiteboard: needtest
Bug Depends on:    
Bug Blocks: 470859    

Description Maxime Porhel CLA 2015-07-07 05:16:50 EDT
This ticket has been created to centralize the correction to do in compartiment support  to correctly handle the workspace image style set on Region/RegionContainer from the VSM or from the tabbar button.
Comment 1 Maxime Porhel CLA 2015-07-07 05:20:58 EDT
Do not forget to remove the ContainerMappingChildrenPresentationConstraints.REGION_REGION_CONTAINER_MAPPING_STYLE_RULE_ID constraint once the image support will be completed.
Comment 2 Maxime Porhel CLA 2015-08-24 09:14:30 EDT
Fist found issue: the Regions are not well located in their RegionContainer when it has a WorkspaceImageDescription: the insets of the content pane seem to be misconfigured.
Comment 3 Eclipse Genie CLA 2015-08-24 10:43:47 EDT
New Gerrit change created: https://git.eclipse.org/r/54409
Comment 4 Eclipse Genie CLA 2015-08-24 10:43:48 EDT
New Gerrit change created: https://git.eclipse.org/r/54408
Comment 5 Eclipse Genie CLA 2015-08-24 11:07:29 EDT
New Gerrit change created: https://git.eclipse.org/r/54416
Comment 6 Maxime Porhel CLA 2015-08-25 05:51:27 EDT
Additional issues corrected by the current: 
 . the border size/color changes are managed only for the FlatContainerStyleDescription 
    -> https://git.eclipse.org/r/#/c/54408/
 . the image needs improvements when workspacePath is or becomes empty to be consistent with the part creation, in the same code area: the gradient style changes are handled only for the freeform DNodeContainer but not for the DNodeList and Regions
    -> https://git.eclipse.org/r/#/c/54409/
 . the corner refresh needs to be update to handle preceding/following regions which does not necessarily support corners (e.g. WorkspaceImage)
    -> https://git.eclipse.org/r/#/c/54416/
Comment 9 Eclipse Genie CLA 2015-08-26 04:13:43 EDT
New Gerrit change created: https://git.eclipse.org/r/54565
Comment 10 Eclipse Genie CLA 2015-08-26 05:44:50 EDT
New Gerrit change created: https://git.eclipse.org/r/54571
Comment 11 Eclipse Genie CLA 2015-08-26 12:04:14 EDT
New Gerrit change created: https://git.eclipse.org/r/54601
Comment 12 Eclipse Genie CLA 2015-08-27 04:08:40 EDT
New Gerrit change created: https://git.eclipse.org/r/54660
Comment 18 Maxime Porhel CLA 2015-10-13 09:54:47 EDT
Validation can be done using the compartment junit data test located in org.eclipse.sirius.tests.junit/data/unit/compartments

Create a project named DesignerTestProject and paste the data in it.


Open ImageDiag (it allows to check the behavior withimage styles defined in the VSM)
Check that image (Regions and RegionContainer) are displayed but also the VSM defined borders.
Check that you can use the tabbar button to change the image and then reset the style. 

Open the VStackDiag (and then the HStackDiag), change the styles of Regions and RegionContainers to an image and then reset the style. 

No error should be thrown (or printed in the console) and each step should produce a consistent diagram and layout.
Comment 19 Florian Barbin CLA 2015-10-15 03:31:13 EDT
Verified on Sirius 3.1.0.201510140809
Comment 20 Pierre-Charles David CLA 2015-10-16 09:09:12 EDT
Available in Sirius 3.1.0.