520624 – Snap to grid constraint is not taken into account after "Arrange Linked Border Node" action.

Bug 520624 - Snap to grid constraint is not taken into account after "Arrange Linked Border Node" action.

Summary: Snap to grid constraint is not taken into account after "Arrange Linked Borde...

Status:	CLOSED FIXED

Alias:	None

Product:	Sirius
Classification:	Modeling
Component:	Diagram (show other bugs)
Version:	5.0.0
Hardware:	All All

Importance:	P3 normal (vote)
Target Milestone:	5.1.0
Assignee:	Florian Barbin
QA Contact:	Julien Dupont

URL:
Whiteboard:
Keywords:	triaged

Depends on:
Blocks:

Reported:	2017-08-07 09:59 EDT by Jessy Mallet
Modified:	2017-11-08 03:37 EST (History)
CC List:	3 users (show)

See Also:	Gerrit Change Gerrit Change Gerrit Change Git Commit Git Commit Git Commit Gerrit Change Gerrit Change 521932 Gerrit Change Git Commit Git Commit

Attachments
Test case to use. (7.52 KB, application/x-zip-compressed) 2017-08-07 09:59 EDT, Jessy Mallet	no flags	Details
ScreenShot of the "Arrange Linked Border Nodes" behaviour (64.99 KB, image/png) 2017-08-07 10:14 EDT, Jessy Mallet	no flags	Details
Property of the grid (14.91 KB, image/png) 2017-08-08 04:06 EDT, Jessy Mallet	no flags	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jessy Mallet

2017-08-07 09:59:47 EDT

Created attachment 269712 [details]
Test case to use.

On diagram with "Snap to grid" activated, when user uses "Arrange Linked Border Nodes", border node are arranged but they do not respect "Snap to grid" constraints and they are not aligned with the grid anymore.
"Arrange Linked Bordered Nodes" should take into account "Snap to grid" constraints.


Step to reproduce :
* import in your workspace the modeling project ArrangeBorderNode.zip,
* open representation "new packageDiag",
* make sure that "Snap to Grid" box is marked in properties view of diagram (tab Rulers&Grid) 
* clic on "Arrange Linked Border Nodes" action (icon on the left top of the diagram)
* border Nodes are arranged but they are not snap on grid anymore -> KO
  They should be align on the grid.

Comment 1 Jessy Mallet

2017-08-07 10:14:28 EDT

Created attachment 269713 [details]
ScreenShot of the "Arrange Linked Border Nodes" behaviour

Comment 2 Jessy Mallet

2017-08-08 04:06:20 EDT

Created attachment 269727 [details]
Property of the grid

Comment 3 Florian Barbin

2017-08-09 03:40:16 EDT

Issue accepted and marked as triaged.

Comment 4 Eclipse Genie

2017-08-24 12:10:35 EDT

New Gerrit change created: https://git.eclipse.org/r/103648

Comment 5 Eclipse Genie

2017-08-29 04:44:35 EDT

New Gerrit change created: https://git.eclipse.org/r/103823

Comment 6 Eclipse Genie

2017-08-31 13:56:55 EDT

Gerrit change https://git.eclipse.org/r/103648 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=3171799f0813dd653e380098679a94281e8b64f0

Comment 7 Eclipse Genie

2017-08-31 13:56:56 EDT

Gerrit change https://git.eclipse.org/r/103823 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=ccf51a731694f55faa891e51356ad31d994d1375

Comment 8 Eclipse Genie

2017-08-31 13:56:57 EDT

Gerrit change https://git.eclipse.org/r/102945 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7ca4dc5d53514dc99943483f57df577298f131b8

Comment 9 Eclipse Genie

2017-09-01 10:14:09 EDT

New Gerrit change created: https://git.eclipse.org/r/104180

Comment 10 Eclipse Genie

2017-09-05 12:37:24 EDT

New Gerrit change created: https://git.eclipse.org/r/104380

Comment 11 Eclipse Genie

2017-09-06 12:12:57 EDT

New Gerrit change created: https://git.eclipse.org/r/104587

Comment 12 Eclipse Genie

2017-09-07 10:02:43 EDT

Gerrit change https://git.eclipse.org/r/104380 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=3dcf269ae1797da6a1226b503220b3f32cc2aa16

Comment 13 Eclipse Genie

2017-09-07 10:02:45 EDT

Gerrit change https://git.eclipse.org/r/104587 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=d93fcfdb588651aa7f3803cbc7be08857ba301d6

Comment 14 Pierre-Charles David

2017-11-08 03:37:00 EST

Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0.