From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Fri, 1 Jul 2016 14:01:00 +0000 (+0100)
Subject: JAL-2110 set reasonable bounds to initial split of SplitFrame divider
X-Git-Tag: Release_2_10_0~140^2~5^2~28
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=0da0ed92182d198fd260866ee804eba9035de6b0;p=jalview.git

JAL-2110 set reasonable bounds to initial split of SplitFrame divider
---

diff --git a/src/jalview/gui/SplitFrame.java b/src/jalview/gui/SplitFrame.java
index 617224f..3b96be8 100644
--- a/src/jalview/gui/SplitFrame.java
+++ b/src/jalview/gui/SplitFrame.java
@@ -206,6 +206,7 @@ public class SplitFrame extends GSplitFrame implements SplitContainerI
     final AlignmentI bottomAlignment = bottomViewport.getAlignment();
     boolean topAnnotations = topViewport.isShowAnnotation();
     boolean bottomAnnotations = bottomViewport.isShowAnnotation();
+    // TODO need number of visible sequences here, not #sequences - how?
     int topCount = topAlignment.getHeight();
     int bottomCount = bottomAlignment.getHeight();
     int topCharHeight = topViewport.getViewStyle().getCharHeight();
@@ -223,6 +224,11 @@ public class SplitFrame extends GSplitFrame implements SplitContainerI
             + (bottomAnnotations ? bottomViewport.calcPanelHeight() : 0);
     double ratio = ((double) topHeight) / (topHeight + bottomHeight);
 
+    /*
+     * limit to 0.2 <= ratio <= 0.8 to avoid concealing all sequences
+     */
+    ratio = Math.min(ratio, 0.8d);
+    ratio = Math.max(ratio, 0.2d);
     setRelativeDividerLocation(ratio);
   }