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

Bug 497848

Summary: EdgePart has to disable refresh-visuals during #bendContent().
Product: [Tools] GEF Reporter: Matthias Wienand <matthias.wienand>
Component: GEF ZestAssignee: gef-inbox <gef-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 1.0.0   
Target Milestone: 4.1.0 (Neon.1) M1   
Hardware: All   
OS: All   
Whiteboard:

Description Matthias Wienand CLA 2016-07-13 11:22:35 EDT
Currently, EdgePart#bendContent() queries attributes of the part's Connection visual in order to apply them to its content element. However, changing the content element results in a #refreshVisual() call that applies the attributes of the content element to the Connection visual. Therefore, some values that are queried from the Connection visual within #bendContent() are overwritten by #refreshVisual() before they are queried.

This is the reason for incorrect anchor hints in some situations. It can be fixed by disabling refreshing of visuals at the start of #bendContent() and restoring it at the end.
Comment 1 Matthias Wienand CLA 2016-07-15 07:29:30 EDT
I extended the code so that is-refresh-visual is set to false at the start of #bendContent() and its original value is restored at the end of #bendContent(). The code is published on the R4_0_maintenance and master branches. Therefore, I resolve this ticket as fixed for 4.1.0 M1.