JAL-2491 sort of working overview/find interaction with split panels

[jalview.git] / src / jalview / viewmodel / ViewportRanges.java

diff --git a/src/jalview/viewmodel/ViewportRanges.java b/src/jalview/viewmodel/ViewportRanges.java
index bbdaa9e..a14fc26 100644 (file)
--- a/src/jalview/viewmodel/ViewportRanges.java
+++ b/src/jalview/viewmodel/ViewportRanges.java
@@ -86,6 +86,13 @@ public class ViewportRanges extends ViewportProperties
    */
   public void setStartRes(int res)
   {
+    int oldres = this.startRes;
+    setStartResInternal(res);
+    changeSupport.firePropertyChange("startres", oldres, res);
+  }
+
+  private void setStartResInternal(int res)
+  {
     if (res > al.getWidth() - 1)
     {
       res = al.getWidth() - 1;
@@ -94,17 +101,14 @@ public class ViewportRanges extends ViewportProperties
     {
       res = 0;
     }
-
-    int oldres = this.startRes;
     this.startRes = res;
-    // changeSupport.firePropertyChange("startres", oldres, res);
   }
 
   public void setStartEndRes(int startres, int endres)
   {
     int oldres = this.startRes;
-    setStartRes(startres);
-    setEndRes(endres);
+    setStartResInternal(startres);
+    setEndResInternal(endres);
     changeSupport.firePropertyChange("startres", oldres, startres);
   }
 
@@ -116,6 +120,14 @@ public class ViewportRanges extends ViewportProperties
    */
   public void setEndRes(int res)
   {
+    int oldres = this.endRes;
+    setEndResInternal(res);
+
+    changeSupport.firePropertyChange("endres", oldres, res);
+  }
+
+  private void setEndResInternal(int res)
+  {
     if (res >= al.getWidth())
     {
       res = al.getWidth() - 1;
@@ -124,10 +136,7 @@ public class ViewportRanges extends ViewportProperties
     {
       res = 0;
     }
-    int oldres = this.endRes;
     this.endRes = res;
-
-    // changeSupport.firePropertyChange("endres", oldres, res);
   }
 
   /**
@@ -138,6 +147,13 @@ public class ViewportRanges extends ViewportProperties
    */
   public void setStartSeq(int seq)
   {
+    int oldseq = this.startSeq;
+    setStartSeqInternal(seq);
+    changeSupport.firePropertyChange("startseq", oldseq, seq);
+  }
+
+  private void setStartSeqInternal(int seq)
+  {
     if (seq > al.getHeight() - 1)
     {
       seq = al.getHeight() - 1;
@@ -146,17 +162,14 @@ public class ViewportRanges extends ViewportProperties
     {
       seq = 0;
     }
-
-    int oldseq = this.startSeq;
     this.startSeq = seq;
-    // changeSupport.firePropertyChange("startseq", oldseq, seq);
   }
 
   public void setStartEndSeq(int startseq, int endseq)
   {
     int oldseq = this.startSeq;
-    setStartSeq(startseq);
-    setEndSeq(endseq);
+    setStartSeqInternal(startseq);
+    setEndSeqInternal(endseq);
     changeSupport.firePropertyChange("startseq", oldseq, startseq);
   }
 
@@ -168,6 +181,13 @@ public class ViewportRanges extends ViewportProperties
    */
   public void setEndSeq(int seq)
   {
+    int oldseq = this.endSeq;
+    setEndSeqInternal(seq);
+    changeSupport.firePropertyChange("endseq", oldseq, seq);
+  }
+
+  private void setEndSeqInternal(int seq)
+  {
     if (seq >= al.getHeight())
     {
       seq = al.getHeight() - 1;
@@ -176,10 +196,7 @@ public class ViewportRanges extends ViewportProperties
     {
       seq = 0;
     }
-    int oldseq = this.endSeq;
     this.endSeq = seq;
-
-    // changeSupport.firePropertyChange("endseq", oldseq, seq);
   }
 
   /**