Community
Participate
Working Groups
Created attachment 138815 [details] proposed patch If image figure can have special connection anchor such that connection would go through transparent pixels of the image and connect to actual image directly, there will be a problem with set different images for ImageFigure while it has incoming/outgoing connectors. When the new Image is set figure is re-validated, such that if bounds of the figure change then connection anchors will be re-calculated. However, if image has the same size, but just different pixels are transparent, the anchor won't be recalculated Attached patch introduces a mechanism to attach/detach IMageChangedListener to image figures. Also it introduces IIMageFigure interface and AbstractImageFigure such that current API won't break.
Alex, do you recall why we never committed this. Did you still need this change?
Looks good to me. I would suggest updating the copyright to 2010.
Committed for 3.6
Committed patch has 1. Corrected header dates and @since 3.6 2. Also made all methods AbstractImageFigure final - no need to override add / remove listeners as well notify method.