JAL-2388 Renamed viewport position props to viewport ranges

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

diff --git a/src/jalview/viewmodel/ViewportPositionProps.java b/src/jalview/viewmodel/ViewportRanges.java
similarity index 89%
rename from src/jalview/viewmodel/ViewportPositionProps.java
rename to src/jalview/viewmodel/ViewportRanges.java
index 61dd18c..5c8cdb6 100644 (file)
--- a/src/jalview/viewmodel/ViewportPositionProps.java
+++ b/src/jalview/viewmodel/ViewportRanges.java
@@ -23,10 +23,11 @@ package jalview.viewmodel;
 import jalview.datamodel.AlignmentI;
 
 /**
- * Supplies and updates viewport properties relating to position such as: start
- * and end residues and sequences
+ * Embryonic class which: Supplies and updates viewport properties relating to
+ * position such as: start and end residues and sequences; ideally will serve
+ * hidden columns/rows too
  */
-public class ViewportPositionProps extends ViewportProperties
+public class ViewportRanges extends ViewportProperties
 {
   // start residue of viewport
   private int startRes;
@@ -49,7 +50,7 @@ public class ViewportPositionProps extends ViewportProperties
    * @param alignment
    *          the viewport's alignment
    */
-  public ViewportPositionProps(AlignmentI alignment)
+  public ViewportRanges(AlignmentI alignment)
   {
     // initial values of viewport settings
     this.startRes = 0;
@@ -59,12 +60,6 @@ public class ViewportPositionProps extends ViewportProperties
     this.al = alignment;
   }
 
-  // ways to update values
-
-  // ways to notify of changes
-
-  // ways to supply positional information
-
   /**
    * Get alignment width in cols, including hidden cols
    */
@@ -112,9 +107,9 @@ public class ViewportPositionProps extends ViewportProperties
     {
       res = al.getWidth() - 1;
     }
-    else if (res < 1)
+    else if (res < 0)
     {
-      res = 1;
+      res = 0;
     }
     this.endRes = res;
   }
@@ -150,9 +145,9 @@ public class ViewportPositionProps extends ViewportProperties
     {
       seq = al.getHeight() - 1;
     }
-    else if (seq < 1)
+    else if (seq < 0)
     {
-      seq = 1;
+      seq = 0;
     }
     this.endSeq = seq;
   }