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/02/15 22:25:10 UTC
svn commit: r1446780 -
/pivot/branches/2.0.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTableViewSkin.java
Author: rwhitcomb
Date: Fri Feb 15 21:25:10 2013
New Revision: 1446780
URL: http://svn.apache.org/r1446780
Log:
More work on PIVOT-873: "editOnMouseDown" style for TableView: fix logic in SelectMode.NONE case
so this style works right to select the row for editing on first click.
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=1446780&r1=1446779&r2=1446780&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 Fri Feb 15 21:25:10 2013
@@ -1242,11 +1242,10 @@ public class TerraTableViewSkin extends
}
} else {
if (selectMode != TableView.SelectMode.NONE) {
- if (tableView.isRowSelected(rowIndex)) {
- selectIndex = rowIndex;
- } else {
+ if (!tableView.isRowSelected(rowIndex)) {
tableView.setSelectedIndex(rowIndex);
}
+ selectIndex = rowIndex;
}
}
}