Bug 560458 - ELK - Wrong node size when using BundledImage style
Summary: ELK - Wrong node size when using BundledImage style
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 10
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
: 560518 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-02-24 06:17 EST by Laurent Redor CLA
Modified: 2020-05-06 11:08 EDT (History)
2 users (show)

See Also:


Attachments
TestElkLayout.zip (10.63 KB, application/x-zip-compressed)
2020-02-24 06:17 EST, Laurent Redor CLA
no flags Details
currentResult.png (5.19 KB, image/png)
2020-05-06 03:13 EDT, 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-24 06:17:35 EST
Created attachment 281904 [details]
TestElkLayout.zip

The nodes using BundledImage as style are not correctly layouted by ELK. They are too small and the label is displayed with "..." after an arrange.

Steps to reproduce
* Import project TestElkLayout from TestElkLayout.zip
* Open diagram "simpleDiagramWithSVGShape"
* Launch an "Arrange All"
* Expected: The diagram is layouted, the 4 classes are sufficiently large to read the label (without "..." at the end of the label).
Comment 1 Laurent Redor CLA 2020-02-25 11:20:18 EST
*** Bug 560518 has been marked as a duplicate of this bug. ***
Comment 2 Steve Monnier CLA 2020-03-20 05:58:22 EDT
I played this scenario but could not reproduce this scenario anymore. This was probably fixed by another commit (maybe from fixes on lists and containers VStack). When I tested the scenario I was on the master branch, commit 440d15d : [553251] Fix GMFHelper regressions from last commit.
Comment 3 Pierre-Charles David CLA 2020-05-05 10:54:39 EDT
@Laurent: do you think this can be closed considering Steve's comment?
Comment 4 Laurent Redor CLA 2020-05-06 03:13:31 EDT
Created attachment 282720 [details]
currentResult.png
Comment 5 Laurent Redor CLA 2020-05-06 03:16:00 EDT
(In reply to Pierre-Charles David from comment #3)
> @Laurent: do you think this can be closed considering Steve's comment?

No, some steps are missing in the steps to reproduce. Here  is the full scenario:
* Import project TestElkLayout from TestElkLayout.zip
* Open diagram "simpleDiagramWithSVGShape"
* Unpinned all elements
* Disable "snap to grid" and "snap to shape"
* Launch an "Arrange All"
* Expected: The diagram is layouted, the 4 classes are sufficiently large to read the label (without "..." at the end of the label), and not like in "currentResult.png".
Comment 6 Steve Monnier CLA 2020-05-06 11:08:52 EDT
With Laurent's last scenario, I was able to reproduce the issue. Note that you need the ELK plugins in your environment in order to reproduce the issue.