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

Bug 481748

Summary:	Remove 'FX' prefix from classes in FX bundle, preserve it within FX.SWT and FX.JFace.
Product:	[Tools] GEF	Reporter:	Alexander Nyßen <nyssen>
Component:	GEF FX	Assignee:	Alexander Nyßen <nyssen>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	0.2.0
Target Milestone:	4.0.0 / 3.11.0 (Neon) M3
Hardware:	All
OS:	All
Whiteboard:
Bug Depends on:
Bug Blocks:	468804

Description Alexander Nyßen

2015-11-09 12:13:18 EST

As stated in bug #468804, we should remove the 'FX' prefix from the remaining classes in the org.eclipse.gef4.fx bundle (some already omit the prefix). This is, because all provided classes will always be used in an JavaFX context, where the 'FX' prefix does not provide a benefit.

In contrast, the classes provided by the FX.SWT and FX.JFace bundles should retain the 'FX' prefix, as they are used in an SWT/JFace context, where the 'FX' prefix indicates the JavaFX integration.

Comment 1 Alexander Nyßen

2015-11-09 12:33:38 EST

I pushed the following changes to origin/master:

- Renamed IFXAnchor to IAnchor, AbstractFXAnchor to AbstractAnchor, FXChopBoxAnchor to ChopBoxAnchor and FXStaticAnchor to StaticAnchor.
- Renamed FXGeometryNode to GeometryNode.
- Renamed FXConnection to Connection, IFXDecoration to IConnectionDecoration, and IFXConnectionRouter to IConnectionRouter, FXPolyBezierConnectionRouter to PolyBezierConnectionRouter, and FXPolylineConnectionRouter to PolylineConnectionRouter.
- Renamed AbstractFXGesture to AbstractGesture, AbstractFXMouseDragGesture to AbstractMouseDragGesture, AbstractFXPinchSpreadGesture to AbstractPinchSpreadGesture, and AbstractFXRotateGesture to AbstractRotateGesture.
- Split FXUtils into NodeUtils and CursorUtils and moved it into o.e.g4.fx.utils package.
- Renamed ImageViewHoverOverlay to HoverOverlayImageView.

Resolving as fixed in 3.11.0 M3.