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

Bug 354622

Summary: [ActivityDiagram] Object Flows selection prevent selecting other close elements.
Product: [Modeling] Papyrus Reporter: David Rabely <david.rabely>
Component: CoreAssignee: Arthur Daussy <arthur.daussy>
Status: REOPENED --- QA Contact:
Severity: normal    
Priority: P3 CC: agarcdomi, arthur.daussy, faure.tristan, raphael.faudou, toni.siljamaki
Version: 0.8.1   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=401542
Whiteboard:
Attachments:
Description Flags
Patch which fixes the issue on top of trunk (SVN rev 8363) none

Description David Rabely CLA 2011-08-12 09:58:29 EDT
Build Identifier: I20110613-1736

When you select a modelized object Flow, it is difficult to reach the name or to select it in the graphical view of a diagramm. In order to manage it, you must move the link of the object flow.

Reproducible: Always
Comment 1 Arthur Daussy CLA 2011-11-09 12:47:59 EST
This bug has been identified as a GMF bug see:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=363362

*** This bug has been marked as a duplicate of bug 363362 ***
Comment 2 Arthur Daussy CLA 2011-12-08 07:29:41 EST
As GMF communality didn't react on this topic. This bug will be corrected locally.

The correction will be use Extended class of WrappringLabel and to overide the method as follow:

	@Override
	public boolean containsPoint(int x, int y) {
		if (isVisible()){
			return super.containsPoint(x, y);			
		}
		return false;
	}
Comment 3 Arthur Daussy CLA 2011-12-08 11:34:06 EST
Correction has been committed on the trunk.
Waiting to backport on the Trunk.
Comment 4 Antonio Garcia-Dominguez CLA 2012-06-19 05:43:08 EDT
(In reply to comment #3)
> Correction has been committed on the trunk.
> Waiting to backport on the Trunk.

On which revision was the fix committed? I'm currently experiencing this bug, and I'm running the 0.9.0.v201206140824 nightly.
Comment 5 Antonio Garcia-Dominguez CLA 2012-06-19 06:02:39 EDT
(In reply to comment #3)
> Correction has been committed on the trunk.
> Waiting to backport on the Trunk.

The SVN history shows that a few commits have been done for this on the 0.8.x branch. Are there any plans to bring these to trunk in the near future?
Comment 6 Arthur Daussy CLA 2012-06-19 06:08:40 EDT
Hi Antonio,

 We are currently backporting all those feature on the trunk. 
 But we are facing some problems due to changes in the meta model. We hope backporting it before the end of the month.

Regards,
Comment 7 Antonio Garcia-Dominguez CLA 2012-06-19 12:41:01 EDT
Created attachment 217557 [details]
Patch which fixes the issue on top of trunk (SVN rev 8363)

Thanks for the feedback! I was in a bit of a hurry, so I went ahead and made a patch using your above instructions :-). It was easier than I thought, as mostly all Papyrus edit parts seem to use the SimpleLabel subclass of WrappingLabel.
Comment 8 Toni Siljamäki CLA 2013-10-10 08:07:31 EDT
Has this one been fixed = to be closed?