svn commit: r17419 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java

Michiel van der Wulp <[email protected]>
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: mvw
Date: 2009-10-21 23:51:47-0700
New Revision: 17419

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

Log:
Fix for issue 5876: PseudoState (sub)instances have become very small.
Figs that are not resizable do not implement a getMinimumSize() method - and the default one just returns (4, 4). Hence this method shall never be called for such Figs.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java?view=diff&pathrev=17419&r1=17418&r2=17419
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java	2009-10-21 23:51:47-0700
@@ -1002,8 +1002,8 @@
      * but then it can never shrink again 
      * (not even if the calculated minimum size is smaller).<p>
      * 
-     * If the user can not resize the fig, e.g. like the FigActor, 
-     * then we return the minimum size.
+     * If the user can not resize the fig, e.g. like the FigActor or FigFinalState, 
+     * then we return the original size.
      */
     protected void updateBounds() {
         if (!checkSize) {
@@ -1016,7 +1016,7 @@
             bbox.height = Math.max(bbox.height, minSize.height);
             setBounds(bbox.x, bbox.y, bbox.width, bbox.height);
         } else {
-            setBounds(bbox.x, bbox.y, minSize.width, minSize.height);
+            setBounds(bbox.x, bbox.y, bbox.width, bbox.height);
         }
     }

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

To unsubscribe from this discussion, e-mail: [[email protected]].
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.