svn commit: r16344 - trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigPackage.java

Michiel van der Wulp <[email protected]>
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: mvw
Date: 2008-12-12 01:48:35-0800
New Revision: 16344

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

Log:
Comment only.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigPackage.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigPackage.java?view=diff&pathrev=16344&r1=16343&r2=16344
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigPackage.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigPackage.java	2008-12-12 01:48:35-0800
@@ -646,6 +646,12 @@
         }
     }
 
+    /**
+     * A text fig for the body of a a Package 
+     * which does not contain any text, 
+     * but solely exists to trigger a jump to a diagram for
+     * the named package when double clicked.
+     */
     class FigPackageFigText extends ArgoFigText {
         
         /**
@@ -677,9 +683,13 @@
             super(owner, bounds, settings, false);
         }
         
-	/*
-	 * @see java.awt.event.MouseListener#mouseClicked(
-         *         java.awt.event.MouseEvent)
+	/**
+	 * TODO: mvw: Would it not be better if this code 
+	 * would go in startTextEditor(), not overruling mouseClicked(). 
+	 * But we made this fig not editable, 
+	 * to stop it from reacting on key-presses.
+	 * Anyhow - this is a hack - abusing a FigText - GEF does 
+	 * not really support double-clicking on a Fig to trigger some action.
 	 */
 	@Override
 	public void mouseClicked(MouseEvent me) {

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

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.