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> = [
{