Fix: Refactor showContextMenu logic

Author: dchyun

packages/components/src/components/hds/advanced-table/index.ts

@@ -661,10 +661,12 @@ export default class HdsAdvancedTable extends Component<HdsAdvancedTableSignatur
   private _isStickyColumn = (
     column: HdsAdvancedTableColumnType
   ): boolean | undefined => {
-    if (this.hasStickyFirstColumn && column.isFirst) {
-      return true;
-    } else if (this.hasStickyFirstColumn === false) {
-      return false;
+    if (column.isFirst) {
+      if (this.hasStickyFirstColumn) {
+        return true;
+      } else if (this.hasStickyFirstColumn === false) {
+        return false;
+      }
     }
     return undefined;
   };

packages/components/src/components/hds/advanced-table/th-sort.ts

@@ -101,13 +101,9 @@ export default class HdsAdvancedTableThSort extends Component<HdsAdvancedTableTh
   }
 
   get showContextMenu(): boolean {
-    const { column, isStickyColumn, hasResizableColumns } = this.args;
+    const { hasResizableColumns, isStickyColumn } = this.args;
 
-    return (
-      (hasResizableColumns ||
-        (isStickyColumn !== undefined && column && column.isFirst)) ??
-      false
-    );
+    return (hasResizableColumns || isStickyColumn !== undefined) ?? false;
   }
 
   get classNames(): string {

packages/components/src/components/hds/advanced-table/th.ts

@@ -144,13 +144,9 @@ export default class HdsAdvancedTableTh extends Component<HdsAdvancedTableThSign
   }
 
   get showContextMenu(): boolean {
-    const { column, isStickyColumn, hasResizableColumns } = this.args;
+    const { hasResizableColumns, isStickyColumn } = this.args;
 
-    return (
-      (hasResizableColumns ||
-        (isStickyColumn !== undefined && column && column.isFirst)) ??
-      false
-    );
+    return (hasResizableColumns || isStickyColumn !== undefined) ?? false;
   }
 
   @action onFocusTrapDeactivate(): void {