svn commit: r18455 - trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java

Author: bobtarling
Date: 2010-06-23 09:30:10-0700
New Revision: 18455

Modified:
   trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java

Log:
Don't reload property panel when moving between tabs of DetailsPane

Modified: trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java?view=diff&pathrev=18455&r1=18454&r2=18455
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/ui/DetailsPane.java	2010-06-23 09:30:10-0700
@@ -508,8 +508,9 @@
         
         // If sel is the ToDo Tab (i.e. is an instance of TabToDoTarget), we 
         // don't need to do anything, because the ToDo Tab is already dealt 
-        // with by it's own listener. 
-        if (!(sel instanceof TabToDoTarget)) {
+        // with by it's own listener. TabProps is also deals with itself.
+        // TODO: Do we really need this for anything?
+        if (!(sel instanceof TabToDoTarget) && !(sel instanceof TabProps)) {
             // The other tabs need to be updated depending on the selection.
             if (sel instanceof TabTarget) {
                 ((TabTarget) sel).setTarget(target);

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

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