From a18fb60524c19ec9d874b050b57d7ab1746fc901 Mon Sep 17 00:00:00 2001
From: t8y2 <1156263951@qq.com>
Date: Mon, 1 Jun 2026 12:35:23 +0800
Subject: [PATCH] fix(structure-editor): hide unsupported column reorder
 controls

---
 .../structure/TableStructureEditor.vue        | 50 ++++++++++---------
 .../tableStructureCapabilities.test.ts        |  1 +
 2 files changed, 28 insertions(+), 23 deletions(-)

diff --git a/apps/desktop/src/components/structure/TableStructureEditor.vue b/apps/desktop/src/components/structure/TableStructureEditor.vue
index dccd31dc0e..09927c69b3 100644
--- a/apps/desktop/src/components/structure/TableStructureEditor.vue
+++ b/apps/desktop/src/components/structure/TableStructureEditor.vue
@@ -467,9 +467,11 @@ function canMoveColumn(index: number, direction: -1 | 1): boolean {
   const targetIndex = index + direction;
   if (targetIndex < 0 || targetIndex >= columns.value.length) return false;
   if (columns.value[index]?.markedForDrop || columns.value[targetIndex]?.markedForDrop) return false;
-  return isCreateMode.value || structureCapabilities.value.reorderColumn;
+  return canShowColumnMoveControls.value;
 }
 
+const canShowColumnMoveControls = computed(() => isCreateMode.value || structureCapabilities.value.reorderColumn);
+
 function moveColumn(index: number, direction: -1 | 1) {
   if (!canMoveColumn(index, direction)) return;
   const targetIndex = index + direction;
@@ -996,28 +998,30 @@ watch(
                   </td>
                   <td :class="structureLastCellClass">
                     <div class="flex items-center gap-1">
-                      <Button
-                        variant="ghost"
-                        size="icon"
-                        :class="structureIconButtonClass"
-                        :disabled="!canMoveColumn(index, -1)"
-                        :title="t('structureEditor.moveColumnUp')"
-                        :aria-label="t('structureEditor.moveColumnUp')"
-                        @click="moveColumn(index, -1)"
-                      >
-                        <ChevronUp :class="structureIconClass" />
-                      </Button>
-                      <Button
-                        variant="ghost"
-                        size="icon"
-                        :class="structureIconButtonClass"
-                        :disabled="!canMoveColumn(index, 1)"
-                        :title="t('structureEditor.moveColumnDown')"
-                        :aria-label="t('structureEditor.moveColumnDown')"
-                        @click="moveColumn(index, 1)"
-                      >
-                        <ChevronDown :class="structureIconClass" />
-                      </Button>
+                      <template v-if="canShowColumnMoveControls">
+                        <Button
+                          variant="ghost"
+                          size="icon"
+                          :class="structureIconButtonClass"
+                          :disabled="!canMoveColumn(index, -1)"
+                          :title="t('structureEditor.moveColumnUp')"
+                          :aria-label="t('structureEditor.moveColumnUp')"
+                          @click="moveColumn(index, -1)"
+                        >
+                          <ChevronUp :class="structureIconClass" />
+                        </Button>
+                        <Button
+                          variant="ghost"
+                          size="icon"
+                          :class="structureIconButtonClass"
+                          :disabled="!canMoveColumn(index, 1)"
+                          :title="t('structureEditor.moveColumnDown')"
+                          :aria-label="t('structureEditor.moveColumnDown')"
+                          @click="moveColumn(index, 1)"
+                        >
+                          <ChevronDown :class="structureIconClass" />
+                        </Button>
+                      </template>
                       <Button
                         v-if="column.original"
                         variant="ghost"
diff --git a/packages/app-tests/tableStructureCapabilities.test.ts b/packages/app-tests/tableStructureCapabilities.test.ts
index b8483b5214..ad04785f52 100644
--- a/packages/app-tests/tableStructureCapabilities.test.ts
+++ b/packages/app-tests/tableStructureCapabilities.test.ts
@@ -73,6 +73,7 @@ test("oracle-like databases expose oracle-compatible structure editing capabilit
     assert.equal(caps.createIndex, true);
     assert.equal(caps.dropIndex, true);
     assert.equal(caps.rebuildIndex, true);
+    assert.equal(caps.reorderColumn, false);
     assert.equal(canEditTableStructure(dbType), true);
   }
 });