eclipselabs/passerelle

Palette groups are always open

Closed this issue · 1 comments

What steps will reproduce the problem?
1. Start DAWN with merged Passerelle
2. Open a workflow

What is the expected output? What do you see instead?

The actor groups should by default not be expanded, but they are.

Please use labels and text to provide additional information.

I found the solution to this problem:

diff --git 
a/com.isencia.passerelle.workbench.model.editor.ui/src/main/java/com/isencia/pas
serelle/workbench/model/editor/ui/views/ActorTreeViewerPage.java 
b/com.isencia.passerelle.workbench.model.editor.ui/src/main/java/com/isencia/pas
serelle/workbench/model/editor/ui/views/ActorTreeViewerPage.java
index 8905cb5..c2e7b63 100644
--- 
a/com.isencia.passerelle.workbench.model.editor.ui/src/main/java/com/isencia/pas
serelle/workbench/model/editor/ui/views/ActorTreeViewerPage.java
+++ 
b/com.isencia.passerelle.workbench.model.editor.ui/src/main/java/com/isencia/pas
serelle/workbench/model/editor/ui/views/ActorTreeViewerPage.java
@@ -110,7 +110,9 @@

     final List<PaletteGroup> grps = builder.getRootPaletteGroups();
     for (PaletteGroup grp : grps) {
-        getTreeViewer().expandToLevel(grp, 1);
+       if (grp.isExpanded()) {
+           getTreeViewer().expandToLevel(grp, 1);
+       }

     }

I'll commit this patch later today.


Original issue reported on code.google.com by s.olof.svensson@gmail.com on 20 Mar 2013 at 5:26

I have committed the patch suggested by Erwin:

      if (grp.isExpanded()) {
        getTreeViewer().expandToLevel(grp, 1);
      }
      for(PaletteGroup childGrp : grp.getChildren()) {
        if (childGrp.isExpanded()) {
          getTreeViewer().expandToLevel(childGrp, 1);
        }
      }

Original comment by s.olof.svensson@gmail.com on 20 Mar 2013 at 8:08

  • Changed state: Fixed