svn commit: r16767 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigCompartmentBox.java

Author: bobtarling
Date: 2009-02-04 12:55:03-0800
New Revision: 16767

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

Log:
Comment only

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigCompartmentBox.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigCompartmentBox.java?view=diff&pathrev=16767&r1=16766&r2=16767
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigCompartmentBox.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigCompartmentBox.java	2009-02-04 12:55:03-0800
@@ -234,6 +234,13 @@
          * so that we can put the last one in edit mode: 
          * This fixes issue 5439. */
         ((FigEditableCompartment) fg).populate();
+        // TODO: The above populate works but seems rather heavy here.
+        // I can see something like this is needed though as events
+        // won't manage this quick enough. Could we make
+        // FigEditableCompartment.createModelElement() create
+        // the new child Fig instance? It may also be useful
+        // for it to return the new model element rather than
+        // the current void return - Bob.
         List figList = fg.getFigs();
         if (figList.size() > 0) {
             Fig fig = (Fig) figList.get(figList.size() - 1);

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

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