Changeset 6249

Timestamp:

2003-12-12T17:49:32+13:00 (20 years ago)

Author:

jmt12

Message:

In order to fix 203B240 it was necessary to force DragTree's UI not to use the buggy VariableHeightLayoutCache

File:

• trunk/gli/src/org/greenstone/gatherer/gui/tree/DragTree.java (modified) (3 diffs)

trunk/gli/src/org/greenstone/gatherer/gui/tree/DragTree.java

@@ -10 +10 @@
 import javax.swing.*;
 import javax.swing.event.*;
+import javax.swing.plaf.basic.BasicTreeUI;
 import javax.swing.tree.*;
 import org.greenstone.gatherer.Dictionary;
@@ -97 +98 @@
     this.setOpaque(true);
     this.setSelectionModel(new DragTreeSelectionModel(this, mixed_selection));
+    this.setShowsRootHandles(true);
+    this.setUI(new BasicTreeUI());
     // Connection
     addTreeSelectionListener(this);
@@ -104 +107 @@
     //renderer.setBackgroundNonSelectionColor(new Color(0,0,0,0));
     setCellRenderer(renderer);
-
-          // Drag'n'drop Setup
-          // Drag source setup.
+    // It turns out VariableHeightLayoutCache is buggy under MacOS, so I'll force it to use FixedHeightLayoutCache. To do that I have to set a cell height, ala below, and set the large model property to true, ala above. And buggy VariableHeightLayoutCache goes away. Plus this actually provides a minor performance boost as the layout manager doesn't have to calculate new bounds each time (well... I suppose any gain is actually pretty much eclipsed by the time taken for file access while we determine what a certain nodes children are).
+    // And once we have the cell renderer, use it to determine a fixed height for the rows
+    Component prototype_row = renderer.getTreeCellRendererComponent(this, "Prototype", true, true, false, 0, true);
+    this.setRowHeight(prototype_row.getSize().height);
+    prototype_row = null;
+
+
+    // Drag'n'drop Setup
+    // Drag source setup.
     DragSource.getDefaultDragSource().createDefaultDragGestureRecognizer(this, drag_action, this);
     // Drop destination setup.