From 569d9bd9d6a270e8ede5867ff0b1f13475047e8b Mon Sep 17 00:00:00 2001
From: Antti Villberg <antti.villberg@semantum.fi>
Date: Mon, 15 Jun 2020 08:36:01 +0300
Subject: [PATCH] TextGrid changes for Solvo

Change-Id: Ibccd48e98a68ce15cb2d9735ae9198f08388112d
---
 .../diagram/elements/TextGridNode.java        | 49 ++++++++++++++-----
 .../simantics/diagram/elements/TextNode.java  |  2 +-
 2 files changed, 39 insertions(+), 12 deletions(-)

diff --git a/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextGridNode.java b/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextGridNode.java
index 3b8de9092..afd9772db 100644
--- a/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextGridNode.java
+++ b/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextGridNode.java
@@ -11,15 +11,8 @@
  *******************************************************************************/
 package org.simantics.diagram.elements;
 
-import gnu.trove.map.TIntObjectMap;
-import gnu.trove.map.TMap;
-import gnu.trove.map.hash.THashMap;
-import gnu.trove.map.hash.TIntObjectHashMap;
-import gnu.trove.procedure.TObjectProcedure;
-
 import java.awt.Color;
 import java.awt.Font;
-import java.awt.Graphics2D;
 import java.awt.geom.AffineTransform;
 import java.awt.geom.Point2D;
 import java.awt.geom.Rectangle2D;
@@ -42,6 +35,12 @@ import org.simantics.scenegraph.g2d.nodes.Decoration;
 import org.simantics.scenegraph.utils.NodeUtil;
 import org.simantics.scl.runtime.function.Function1;
 
+import gnu.trove.map.TIntObjectMap;
+import gnu.trove.map.TMap;
+import gnu.trove.map.hash.THashMap;
+import gnu.trove.map.hash.TIntObjectHashMap;
+import gnu.trove.procedure.TObjectProcedure;
+
 /**
  * TODO: does not work remotely, e.g. rowIds 
  */
@@ -264,13 +263,41 @@ public class TextGridNode extends G2DParentNode implements Decoration {
         return rowIds.get(y);
     }
     
+    class FontSizeSum implements TObjectProcedure<Cell> {
+        
+        final double[] hs;
+        
+        FontSizeSum(int maxY) {
+            hs = new double[maxY+1];
+        }
+        
+        @Override
+        public boolean execute(Cell key) {
+            A a = nodes.get(key);
+            int h = a.getFont().getSize();
+            if(h > hs[key.y])
+                hs[key.y] = h;
+            return true;
+        }
+        
+        public double getH() {
+            double result = 0;
+            for(int i=0;i<hs.length;i++)
+                result += hs[i];
+            return result;
+        }
+        
+    }
+
     @Override
     public void refresh() {
+        FontSizeSum fss = new FontSizeSum(computeRows());
+        nodes.forEachKey(fss);
         Vec2d delta = getDelta(FACTOR);
         if(delta != null)
-            setTransform(AffineTransform.getTranslateInstance(delta.x, delta.y-2.1*computeRows()*(up ? 1.0 : 0.0)));
+            setTransform(AffineTransform.getTranslateInstance(delta.x, delta.y-0.15*fss.getH()*(up ? 1.0 : 0.0)));
         else
-            setTransform(AffineTransform.getTranslateInstance(0, -2.1*computeRows()*(up ? 1.0 : 0.0)));
+            setTransform(AffineTransform.getTranslateInstance(0, -0.15*fss.getH()*(up ? 1.0 : 0.0)));
         super.refresh();
     }
     
@@ -350,8 +377,8 @@ public class TextGridNode extends G2DParentNode implements Decoration {
     	Rectangle2D bounds = super.getBoundsInLocal(false);
     	if(bounds == null) return false;
     	Point2D localPos = NodeUtil.worldToLocal(this, event.controlPosition, new Point2D.Double());
-    	double x = localPos.getX();
-    	double y = localPos.getY()+2.1*computeRows()*(up ? 1.0 : 0.0);
+        double x = localPos.getX();
+    	double y = localPos.getY()+(up ? bounds.getHeight() : 0.0);
     	boolean hit = bounds.contains(x, y);
     	return hit;
     	
diff --git a/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextNode.java b/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextNode.java
index bf1bf563b..7c684f30b 100644
--- a/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextNode.java
+++ b/bundles/org.simantics.diagram/src/org/simantics/diagram/elements/TextNode.java
@@ -632,7 +632,7 @@ public class TextNode extends G2DNode implements IDynamicSelectionPainterNode, L
         boolean hover = hasState(STATE_HOVER);
         boolean editing = hasState(STATE_EDITING);
 
-        if (!isSelected && hover) {
+        if (!isSelected && hover && textListener != null) {
             color = add(color, 120, 120, 120);
         }
 
-- 
2.47.1