WB Attachments: Fix moving attachments column (#7390)

* Use visual column index of attachments column

* Lint code with ESLint and Prettier

Triggered by 35eca19 on branch refs/heads/issue-7389

Author: alesan99

specifyweb/frontend/js_src/lib/components/WorkBench/WbAttachmentsPreview.tsx

@@ -173,7 +173,7 @@ function fetchRowAttachments(
   ) => void
 ): void {
   // Look for Attachments column
-  const attachmentColumnIndex = getAttachmentsColumn(dataset);
+  const attachmentColumnIndex = hot.toVisualColumn(getAttachmentsColumn(dataset));
   if (attachmentColumnIndex === -1) return;
 
   // Each row should have comma-separated IDs for SpDataSetAttachments

specifyweb/frontend/js_src/lib/components/WorkBench/attachmentHelpers.tsx

@@ -73,6 +73,9 @@ export function usesAttachments(dataset: Dataset): boolean {
   return dataset.columns.includes(ATTACHMENTS_COLUMN);
 }
 
+/**
+ * Returns the *physical* column index of the attachment JSON data.
+ */
 export function getAttachmentsColumn(dataset: Dataset): number {
   if (!usesAttachments(dataset)) {
     return -1;

specifyweb/frontend/js_src/lib/components/WorkBench/handsontable.ts

@@ -226,8 +226,13 @@ function setColumnWidths(hot: Handsontable, dataset: Dataset): void {
    * For simplicity, the width is limited to 100px to reflect the likely shorter displayed text.
    */
   const attachmentColumnMaxWidth = 100;
-  const attachmentsColumnIndex = getAttachmentsColumn(dataset);
-  if (attachmentsColumnIndex !== -1) {
+  const attachmentsColumnIndex = hot.toVisualColumn(
+    getAttachmentsColumn(dataset)
+  );
+  if (
+    attachmentsColumnIndex !== -1 &&
+    hot.getColWidth(attachmentsColumnIndex) > attachmentColumnMaxWidth
+  ) {
     colWidths = [];
     colWidths[attachmentsColumnIndex] = Math.min(
       hot.getColWidth(attachmentsColumnIndex),