You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by rw...@apache.org on 2013/03/07 16:01:19 UTC

svn commit: r1453906 - /pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java

Author: rwhitcomb
Date: Thu Mar  7 15:01:19 2013
New Revision: 1453906

URL: http://svn.apache.org/r1453906
Log:
Fix the "getRowAt" method in TerraTableViewSkin so that the
variable row height case works the same as fixed height:
return -1 if the y value is beyond the end of the table
otherwise return the correct row.

Modified:
    pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java

Modified: pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java
URL: http://svn.apache.org/viewvc/pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java?rev=1453906&r1=1453905&r2=1453906&view=diff
==============================================================================
--- pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java (original)
+++ pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java Thu Mar  7 15:01:19 2013
@@ -589,11 +589,11 @@ public class TerraTableViewSkin extends 
             }
         } else {
             rowIndex = (y / (fixedRowHeight + 1));
+        }
 
-            List<Object> tableData = (List<Object>)tableView.getTableData();
-            if (rowIndex >= tableData.getLength()) {
-                rowIndex = -1;
-            }
+        List<Object> tableData = (List<Object>)tableView.getTableData();
+        if (rowIndex >= tableData.getLength()) {
+            rowIndex = -1;
         }
 
         return rowIndex;