You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by tv...@apache.org on 2009/10/04 15:54:09 UTC
svn commit: r821543 -
/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java
Author: tvolkert
Date: Sun Oct 4 13:54:08 2009
New Revision: 821543
URL: http://svn.apache.org/viewvc?rev=821543&view=rev
Log:
Fixed bug in TerraFileBrowserSkin
Modified:
incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java
Modified: incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java
URL: http://svn.apache.org/viewvc/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java?rev=821543&r1=821542&r2=821543&view=diff
==============================================================================
--- incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java (original)
+++ incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraFileBrowserSkin.java Sun Oct 4 13:54:08 2009
@@ -498,6 +498,7 @@
updatingSelection = true;
Sequence<File> files = (Sequence<File>)tableView.getSelectedRows();
+ // TODO Revisit this for loop
for (int i = 0, n = files.getLength(); i < n; i++) {
File file = files.get(i);
files.update(i, file);
@@ -734,18 +735,19 @@
@Override
public void selectedFilesChanged(FileBrowser fileBrowser, Sequence<File> previousSelectedFiles) {
if (!updatingSelection) {
- fileTableView.clearSelection();
-
Sequence<File> selectedFiles = fileBrowser.getSelectedFiles();
+ ArrayList<Span> selectedRanges = new ArrayList<Span>();
for (int i = 0, n = selectedFiles.getLength(); i < n; i++) {
File selectedFile = selectedFiles.get(i);
int index = files.indexOf(selectedFile);
if (index != -1) {
- fileTableView.addSelectedIndex(index);
+ selectedRanges.add(new Span(index, index));
}
}
+
+ fileTableView.setSelectedRanges(selectedRanges);
}
}