svn commit: r16565 - trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram

Author: penyaskito
Date: 2009-01-10 17:25:02-0800
New Revision: 16565

Modified:
   trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigClassifierRole.java
   trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigLifeLine.java

Log:
Issue 5608: Fill color is not applied to the activations of a classifier role.

Modified: trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigClassifierRole.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigClassifierRole.java?view=diff&pathrev=16565&r1=16564&r2=16565
==============================================================================
--- trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigClassifierRole.java	(original)
+++ trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigClassifierRole.java	2009-01-10 17:25:02-0800
@@ -318,6 +318,7 @@
     @Override
     public void setFillColor(Color color) {
         headFig.setFillColor(color);
+        lifeLineFig.setFillColor(color);
     }
     
     @Override

Modified: trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigLifeLine.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigLifeLine.java?view=diff&pathrev=16565&r1=16564&r2=16565
==============================================================================
--- trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigLifeLine.java	(original)
+++ trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/FigLifeLine.java	2009-01-10 17:25:02-0800
@@ -83,9 +83,15 @@
         stackedActivations = createStackedActivations(messages);
         
         for (FigActivation figAct : activations) {
+            // TODO: shouldn't this be done in creation, and
+            // with the info from DiagramSettings?
+            figAct.setFillColor(getFillColor());
             addFig(figAct);
         }
         for (FigActivation figAct : stackedActivations) {
+            // TODO: shouldn't this be done in creation, and
+            // with the info from DiagramSettings?
+            figAct.setFillColor(getFillColor());
             addFig(figAct);
         }       
         calcBounds();

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

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