Use @observable.ref for hoveredStructVarTableRowIds

PR feedback Aaron
cBioPortal · Apr 17, 2023 · 9a1f2cf · 9a1f2cf
1 parent 0b296c6
commit 9a1f2cf
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/pages/studyView/table/StructuralVariantMultiSelectionTable.tsx b/src/pages/studyView/table/StructuralVariantMultiSelectionTable.tsx
@@ -108,7 +108,7 @@ export class StructuralVariantMultiSelectionTable extends React.Component<
         modalPanelName: '',
     };
 
-    @observable private hoveredStructVarTableRowIds: string[] = [];
+    @observable.ref private hoveredStructVarTableRowIds: string[] = [];
 
     public static defaultProps = {
         cancerGeneFilterEnabled: false,
@@ -635,9 +635,15 @@ export class StructuralVariantMultiSelectionTable extends React.Component<
     @action.bound
     onStructVarHover(rowId: string, isHovered: boolean) {
         if (isHovered) {
-            this.hoveredStructVarTableRowIds.push(rowId);
+            this.hoveredStructVarTableRowIds = [
+                rowId,
+                ...this.hoveredStructVarTableRowIds,
+            ];
         } else {
-            _.pull(this.hoveredStructVarTableRowIds, rowId);
+            this.hoveredStructVarTableRowIds = _.without(
+                this.hoveredStructVarTableRowIds,
+                rowId
+            );
         }
     }