chore(table): move workaround comment back to its usage

Esri · Jan 20, 2025 · 22a0be3 · 22a0be3
1 parent 4e7dc05
commit 22a0be3
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/calcite-components/src/components/table/table.tsx b/packages/calcite-components/src/components/table/table.tsx
@@ -69,9 +69,6 @@ export class Table extends LitElement implements LoadableComponent {
 
   @state() pageStartRow = 1;
 
-  /* Workaround for Safari https://bugs.webkit.org/show_bug.cgi?id=258430 https://bugs.webkit.org/show_bug.cgi?id=239478 */
-
-  // ⚠️ browser-sniffing is not a best practice and should be avoided ⚠️
   @state() readCellContentsToAT: boolean;
 
   @state() selectedCount = 0;
@@ -178,7 +175,11 @@ export class Table extends LitElement implements LoadableComponent {
 
   async load(): Promise<void> {
     setUpLoadableComponent(this);
+
+    /* Workaround for Safari https://bugs.webkit.org/show_bug.cgi?id=258430 https://bugs.webkit.org/show_bug.cgi?id=239478 */
+    // ⚠️ browser-sniffing is not a best practice and should be avoided ⚠️
     this.readCellContentsToAT = /safari/i.test(getUserAgentString());
+
     this.listenOn(this.el.shadowRoot, "slotchange", this.handleSlotChange);
   }