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

Bug 493875

Summary:

Edge creation with its border nodes: Problem if diagram has scroll bar

Product:

[Modeling] Sirius

Reporter:

Laurent Redor <laurent.redor>

Component:

Diagram

Assignee:

Laurent Redor <laurent.redor>

Status:

CLOSED FIXED

QA Contact:

Florian Barbin <florian.barbin>

Severity:

normal

Priority:

CC:

florian.barbin, julien.dupont

Version:

3.1.0

Keywords:

triaged

Target Milestone:

4.0.0

Hardware:

OS:

Windows 10

See Also:

https://git.eclipse.org/r/73009
https://git.eclipse.org/r/73252
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=65f8a547f2d58d7668a8ffee087ea5fcc6485283
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=2a0e18d88e1bd806866f4c4e663517f0573a3af7

Whiteboard:

Bug Depends on:

Bug Blocks:

494996

Attachments:

Description	Flags
Edge Creation feedback	none
Edge creation result	none

Description Laurent Redor

2016-05-18 04:44:08 EDT

Since the bug 471139, there is a regression if the diagram has scroll bars.

Steps to reproduce:
* Use the same scenario as 471139 but move "P5-2" and "P5-3" to have horizontal and vertical scroll bars.

Comment 1 Eclipse Genie

2016-05-18 04:50:51 EDT

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

Comment 2 Julien Dupont

2016-05-19 10:09:04 EDT

Created attachment 261859 [details]
Edge Creation feedback

Comment 3 Julien Dupont

2016-05-19 10:09:23 EDT

Created attachment 261860 [details]
Edge creation result

Comment 4 Julien Dupont

2016-05-19 10:12:28 EDT

To reproduce, it's necessary to have space on the left of horizontal scroll bar and on the top of verticla scroll bar as it can see on EdgeCreationResult.png with red circle.

Comment 5 Eclipse Genie

2016-05-20 05:27:43 EDT

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

Comment 6 Eclipse Genie

2016-05-23 04:38:21 EDT

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

Comment 7 Eclipse Genie

2016-05-23 09:56:42 EDT

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

Comment 8 Laurent Redor

2016-05-23 09:58:02 EDT

Fixed in Sirius 4.0.0 RC2

Comment 9 Florian Barbin

2016-05-26 11:34:57 EDT

Verified on Sirius 4.0.0.RC2

Comment 10 Pierre-Charles David

2016-06-24 08:03:07 EDT

Available in Sirius 4.0.0.