Bug 560559 - [ELK] Wrong layout for container with regions (vertical stack or horizontal stack)
Summary: [ELK] Wrong layout for container with regions (vertical stack or horizontal s...
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Laurent Redor CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2020-02-26 09:36 EST by Laurent Redor CLA
Modified: 2020-03-20 05:25 EDT (History)
1 user (show)

See Also:


Attachments
TestElkLayout.zip (13.18 KB, application/x-zip-compressed)
2020-02-26 09:36 EST, Laurent Redor CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Laurent Redor CLA 2020-02-26 09:36:21 EST
Created attachment 281939 [details]
TestElkLayout.zip

The ELK layout wrongly set with invalid values for the coordinates of the regions. For example, in vertical stack, the first region is not at the beginning of the list. As soon as this wrong values are set there are unexpected behaviors in Sirius (resize, move, ...).

Steps to reproduce:
* Import project TestElkLayout from TestElkLayout.zip
* Open diagram "diagramWithRegions"
* Launch an "arrange all"
* Expected: 
    * The list container is correctly resized (fit to list items) and not too high.
	* The container with vertical layout, "root_V", is correctly resized and its children are correctly located.
	    * Select the region "MyClass1", the y coordinate should be "0" (ie at the top of the list, ie without space bewteen the title "root_V" and the list).
	* The container with horizontal layout, "root_H", is correctly resized and its children are correctly located
	    * Select the region "MyClass1", the y coordinate should be "0" (ie at the top of the list, ie without space bewteen the title "root_V" and the list) and the height should fulfill the list space.
Comment 1 Eclipse Genie CLA 2020-03-11 08:59:35 EDT
New Gerrit change created: https://git.eclipse.org/r/159182
Comment 2 Eclipse Genie CLA 2020-03-11 08:59:36 EDT
New Gerrit change created: https://git.eclipse.org/r/159181
Comment 5 Laurent Redor CLA 2020-03-15 16:07:53 EDT
Only container with VStack and empty regions are handled by above commits.