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

Bug 519044

Summary: Zoom modify edge display in diagram.
Product: [Modeling] Sirius Reporter: Jessy Mallet <jessy.mallet>
Component: DiagramAssignee: Jessy Mallet <jessy.mallet>
Status: CLOSED FIXED QA Contact: Julien Dupont <julien.dupont>
Severity: normal    
Priority: P3 CC: julien.dupont, laurent.redor, pierre-charles.david
Version: 5.0.0Keywords: triaged
Target Milestone: 5.1.0   
Hardware: PC   
OS: Windows NT   
See Also: https://git.eclipse.org/r/100485
https://git.eclipse.org/r/100933
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=758147ad8d5b123304b7e90fc2295c79fde165f6
https://git.eclipse.org/r/101884
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b55ab983ba646eba0f14693aef048bbcfc86711b
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=51684694d9f3d47058bbe56094cbdfffc3256ee5
https://git.eclipse.org/r/102475
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=f14ec1fcce397ee670f045c06fa89d91ba1bb663
https://git.eclipse.org/r/102697
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=f67e3101a97c87042aa77bd1bc267ad414278853
Whiteboard:
Attachments:
Description Flags
Test case to use in order to reproduce error.
none
Display edge with zoom 75 before fix
none
Display edge with zoom 50 before fix
none
Display edge with zoom 50 after fix none

Description Jessy Mallet CLA 2017-06-30 11:57:14 EDT
Created attachment 269149 [details]
Test case to use in order to reproduce error.

On diagram using rectilinear edges and bordered nodes, zoom in/out modify number of segments composing the rectilinear edge.
Even the diagram is not modified, the display is different.

Step to reproduce :

- import in your workspace the modeling project RectilinearEdgeCreation.zip,
- open diagrams "new packageDiag"
- use tool "createEdge" to create rectilinear edge between the center of the class 
C1 of the P3 package and the top left corner of the class C1 of package P4.
- Zoom to 75%
- the display of edge has changed -> KO
The edge should be always composed of 3 segments.
Comment 1 Jessy Mallet CLA 2017-06-30 11:57:58 EDT
Created attachment 269150 [details]
Display edge with zoom 75 before fix
Comment 2 Jessy Mallet CLA 2017-06-30 11:58:34 EDT
Created attachment 269151 [details]
Display edge with zoom 50 before fix
Comment 3 Jessy Mallet CLA 2017-06-30 11:59:44 EDT
Created attachment 269152 [details]
Display edge with zoom 50 after fix
Comment 4 Eclipse Genie CLA 2017-07-07 12:11:01 EDT
New Gerrit change created: https://git.eclipse.org/r/100933
Comment 6 Eclipse Genie CLA 2017-07-25 04:53:35 EDT
New Gerrit change created: https://git.eclipse.org/r/101884
Comment 9 Eclipse Genie CLA 2017-08-03 12:14:08 EDT
New Gerrit change created: https://git.eclipse.org/r/102475
Comment 11 Eclipse Genie CLA 2017-08-08 10:49:54 EDT
New Gerrit change created: https://git.eclipse.org/r/102697
Comment 13 Pierre-Charles David CLA 2017-08-16 09:38:03 EDT
It seems we're done. @Jessy, please reopen if you see any remaining tasks (doc, tests, etc.) that I'm unaware of, and list them.
Comment 14 Pierre-Charles David CLA 2017-11-08 03:36:33 EST
Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0.