svn commit: r16361 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java

Author: mvw
Date: 2008-12-15 01:35:21-0800
New Revision: 16361

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java

Log:
Comments + worked out a minor TODO.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java?view=diff&pathrev=16361&r1=16360&r2=16361
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/CompartmentFigText.java	2008-12-15 01:35:21-0800
@@ -38,8 +38,10 @@
 import org.tigris.gef.presentation.FigText;
 
 /**
- * A FigText class extension for FigClass/FigInterface/FigUseCase
- * compartments.<p>
+ * A single line FigText class extension for editable 
+ * FigClass/FigInterface/FigUseCase
+ * compartments that use notation. 
+ * When selected, this compartment is highlighted.<p>
  *
  * This implementation now supports the extension point compartment in
  * a use case.<p>
@@ -63,6 +65,7 @@
      * @deprecated for 0.27.3 by tfmorris.  Only used for line color which we
      * can get from the render settings or GEF.
      */
+    @Deprecated
     private Fig refFig;
 
     /**
@@ -303,7 +306,7 @@
      * This is actually used to mark this Fig as selected, however setSelected
      * is set final in GEF.
      * TODO: Can setSelected be used without side-effect if GEF is adjusted?
-     * Otherwise consider renamaing as setSelectedChild and try to make
+     * Otherwise consider renaming as setSelectedChild and try to make
      * protected.
      * @param flag  <code>true</code> if the entry is to be highlighted,
      *              <code>false</code> otherwise.
@@ -314,7 +317,7 @@
     
     /**
      * Extends the normal paint function in order to display a similar
-     * selectionbox to that given for a non-resizable FigNode.
+     * selection-box to that given for a non-resizable FigNode.
      * @param g the graphics object
      * @see org.tigris.gef.presentation.FigText#paint(java.awt.Graphics)
      */
@@ -373,12 +376,9 @@
     
     /**
      * Called when text editing has completed on this Fig.
-     * TODO: This should become protected once textEdited in FigEnumeration
-     * has been refactored to FigNodeModelElement
      */
-    public void textEdited() {
+    protected void textEdited() {
         setHighlighted(true);
-        getNotationProvider().parse(getOwner(), getText());
-        setText(getNotationProvider().toString(getOwner(), getNpArguments()));
+        super.textEdited();
     }
 }

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=984320

To unsubscribe from this discussion, e-mail: [[email protected]].