521802 – Location is wrong after a drag and drop from a container to the diagram

Bug 521802 - Location is wrong after a drag and drop from a container to the diagram

Summary: Location is wrong after a drag and drop from a container to the diagram

Status:	NEW

Alias:	None

Product:	Sirius
Classification:	Modeling
Component:	Diagram (show other bugs)
Version:	4.1.0
Hardware:	PC Windows NT

Importance:	P3 normal (vote)
Target Milestone:	---
Assignee:	Project Inbox
QA Contact:

URL:
Whiteboard:
Keywords:	triaged

Depends on:
Blocks:

Reported:	2017-09-04 03:55 EDT by Laurent Redor
Modified:	2017-12-28 04:35 EST (History)
CC List:	2 users (show)

See Also:	520629 Gerrit Change Gerrit Change Git Commit Git Commit

Attachments
DragNDrop.zip (3.99 KB, application/x-zip-compressed) 2017-09-04 03:55 EDT, Laurent Redor	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 Laurent Redor

2017-09-04 03:55:01 EDT

Created attachment 270061 [details]
DragNDrop.zip

If you drag'n'drop an element from a container to the diagram, the element is not dropped at the location where you drop the element. On the other hand, the feedback is correct.

The element seems to be dropped at the center of the visible part of the diagram.

Steps to reproduce:
* Import project DragNDrop from DragNDrop.zip (project created with ODCE 9.1.1/Sirius 4.1.3)
* Open diagram "new TC1041 representation 2 Blank"
* Drag'n'drop P2 from P1 to the diagram (at the top left of the diagram)
* KO: P2 is dropped at the center of the visible part of the diagram

This problem is true with or without snapToGrid enabled. It has been discovered during the test of bug 520629.

Comment 1 Eclipse Genie

2017-09-05 05:06:25 EDT

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

Comment 2 Eclipse Genie

2017-09-05 07:57:44 EDT

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

Comment 3 Florian Barbin

2017-09-06 09:28:44 EDT

TODO: reactivate the checkEditPartLocation in DragNDropTest

Comment 4 Eclipse Genie

2017-09-06 12:35:37 EDT

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

Comment 5 Eclipse Genie

2017-09-06 12:35:40 EDT

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

Comment 6 Pierre-Charles David

2017-12-28 04:35:41 EST

Marking as triaged as there has already been some tests changes committed on this, so I assume the issue is real and reproducible in these tests.