Table: Return index on appendRow

yamcs · Oct 19, 2023 · 9df7408 · 9df7408
1 parent 533c6b9
commit 9df7408
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/scripting/SpreadSheetTableWrapper.ts b/src/scripting/SpreadSheetTableWrapper.ts
@@ -6,9 +6,9 @@ export class SpreadSheetTableWrapper {
   constructor(
     private spreadsheet: SpreadSheetTable,
     private scriptEngine: ScriptEngine
-  ) {}
+  ) { }
 
-  addModifiedListener(listener: any) {}
+  addModifiedListener(listener: any) { }
 
   addSelectionChangedListener(listener: any) {
     this.spreadsheet.addSelectionChangedListener(() => {
@@ -20,7 +20,7 @@ export class SpreadSheetTableWrapper {
   }
 
   appendRow() {
-    this.spreadsheet.appendRow();
+    return this.spreadsheet.appendRow();
   }
 
   deleteColumn(index: number) {
@@ -98,4 +98,9 @@ export class SpreadSheetTableWrapper {
   setRowForeground(row: number, color: Color) {
     this.spreadsheet.setRowForeground(row, color);
   }
+
+  revealRow(index: number) {
+    // Do nothing for now.
+    // Method added to not crash if it is used.
+  }
 }
diff --git a/src/widgets/others/Table.ts b/src/widgets/others/Table.ts
@@ -36,6 +36,7 @@ export class Table extends Widget {
   init() {
     this.tableWrapper = document.createElement("div");
     this.tableWrapper.style.display = "none";
+    this.tableWrapper.style.overflow = "auto";
     this.table = document.createElement("table");
     this.table.style.tableLayout = "fixed";
     this.table.style.borderSpacing = "0";