RED

refs 2490

org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/extension/ITutorialStepUI.java

@@ -38,6 +38,8 @@ public interface ITutorialStepUI extends ITutorialStep {
 	/**
 	 * Creates the composite to be shown in the editor for the given tutorial step. Sub-classes may
 	 * override. The default shows the tutorial step title.
+	 * TODO(VA) Which default? The one of the
+	 * base? or is it provided by the service?
 	 */
 	void createDetailsComposite(Composite parent);
 }

org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/extension/base/PropertySectionBase.java

@@ -289,6 +289,9 @@ public abstract class PropertySectionBase extends AbstractPropertySection {
 		dbc = new EMFDataBindingContext();
 	}
 
+	// TODO(VA) I thought you were giving up using data bindings, are then the following new methods
+	// necessary?
+
 	/**
 	 * Binds a positive {@link BigDecimal} value with the default number of a maximum of three
 	 * fraction digits to be used in the String representation.

org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/extension/base/TutorialStepUIAtomicWithWhitelistBase.java

@@ -71,4 +71,7 @@ public abstract class TutorialStepUIAtomicWithWhitelistBase extends TutorialStep
 	public boolean contextMenuContributionVisible(EObject element, IContributionItem contribution) {
 		return false;
 	}
+
+	// TODO(VA) So by default everything is false? Isn't it more of a black list than a white list
+	// then?
 }