fix: tree and rollup default to non-sortable (#1404)

Closes #1402
deephaven · Jul 10, 2023 · 5a8f34d · 5a8f34d
1 parent ba13c91
commit 5a8f34d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/iris-grid/src/IrisGridTableModel.ts b/packages/iris-grid/src/IrisGridTableModel.ts
@@ -269,10 +269,6 @@ class IrisGridTableModel extends IrisGridTableModelTemplate<Table, UIRow> {
     return this.frozenColumns.includes(this.columns[modelIndex].name);
   }
 
-  isColumnSortable(modelIndex: ModelIndex): boolean {
-    return this.columns[modelIndex].isSortable ?? true;
-  }
-
   async delete(ranges: GridRange[]): Promise<void> {
     if (!this.isDeletableRanges(ranges)) {
       throw new Error(`Undeletable ranges ${ranges}`);

diff --git a/packages/iris-grid/src/IrisGridTableModelTemplate.ts b/packages/iris-grid/src/IrisGridTableModelTemplate.ts
@@ -1581,6 +1581,10 @@ class IrisGridTableModelTemplate<
     return !this.isKeyColumn(modelIndex);
   }
 
+  isColumnSortable(modelIndex: ModelIndex): boolean {
+    return this.columns[modelIndex].isSortable ?? true;
+  }
+
   isKeyColumn(x: ModelIndex): boolean {
     return x < (this.inputTable?.keyColumns.length ?? 0);
   }