This is an automated email from the ASF dual-hosted git repository.

nicholasjiang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/paimon-webui.git


The following commit(s) were added to refs/heads/main by this push:
     new b954e27d [Bugfix] Fix mobile field order (#358)
b954e27d is described below

commit b954e27dcde28d5b36894a7c820ac100f0d47283
Author: s7monk <[email protected]>
AuthorDate: Thu Jun 13 13:50:00 2024 +0800

    [Bugfix] Fix mobile field order (#358)
---
 .../src/views/metadata/components/columns-form/index.tsx          | 8 ++++++++
 .../src/views/metadata/components/table-column-content/index.tsx  | 6 ------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/paimon-web-ui/src/views/metadata/components/columns-form/index.tsx 
b/paimon-web-ui/src/views/metadata/components/columns-form/index.tsx
index 2759484d..886e94ca 100644
--- a/paimon-web-ui/src/views/metadata/components/columns-form/index.tsx
+++ b/paimon-web-ui/src/views/metadata/components/columns-form/index.tsx
@@ -72,10 +72,18 @@ export default defineComponent({
       props.onConfirm!()
     }
 
+    function handleFieldSort() {
+      const columns = toRaw(formValue.value).tableColumns
+      columns.forEach((item, index) => {
+        item.sort = index
+      })
+    }
+
     async function handleEditColumn() {
       await formRef.value.validate()
       const currentTable = toRaw(catalogStore.currentTable)!
       const { catalogName, databaseName, tableName } = currentTable
+      handleFieldSort()
       await editColumn({
         params: {
           catalogName,
diff --git 
a/paimon-web-ui/src/views/metadata/components/table-column-content/index.tsx 
b/paimon-web-ui/src/views/metadata/components/table-column-content/index.tsx
index 06843067..fd0d44c8 100644
--- a/paimon-web-ui/src/views/metadata/components/table-column-content/index.tsx
+++ b/paimon-web-ui/src/views/metadata/components/table-column-content/index.tsx
@@ -60,12 +60,6 @@ export default defineComponent({
       props.modelValue.splice(i, 1)
     }
 
-    watch(props.modelValue, (newData) => {
-      newData.forEach((item, index) => {
-        item.sort = index
-      })
-    }, { deep: true })
-
     const columns = computed(() => {
       const baseColumns: DataTableColumns<ColumnDTO> = [
         {

Reply via email to