From dc5b7ab9f75a3e34af149b352538d9f286729b97 Mon Sep 17 00:00:00 2001 From: braks <78412429+bcakmakoglu@users.noreply.github.com> Date: Sat, 30 Jul 2022 13:22:41 +0200 Subject: [PATCH] fix(gui-v2): previous and next item swapped --- .../components/smartsheet/sidebar/MenuTop.vue | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/nc-gui-v2/components/smartsheet/sidebar/MenuTop.vue b/packages/nc-gui-v2/components/smartsheet/sidebar/MenuTop.vue index 130a33c14c..4aaa817534 100644 --- a/packages/nc-gui-v2/components/smartsheet/sidebar/MenuTop.vue +++ b/packages/nc-gui-v2/components/smartsheet/sidebar/MenuTop.vue @@ -80,18 +80,18 @@ function initializeSortable(el: HTMLElement) { const currentItem: Record = views.value[oldIndex] // get items meta of before and after the moved item - const nextItem: Record = views.value[newIndex] - const previousItem: Record = views.value[newIndex + 1] + const previousItem: Record = views.value[newIndex] + const nextItem: Record = views.value[newIndex + 1] let nextOrder: number // set new order value based on the new order of the items if (views.value.length - 1 === newIndex) { - nextOrder = parseFloat(nextItem.order) + 1 + nextOrder = parseFloat(previousItem.order) + 1 } else if (newIndex === 0) { - nextOrder = parseFloat(nextItem.order) / 2 + nextOrder = parseFloat(previousItem.order) / 2 } else { - nextOrder = (parseFloat(nextItem.order) + parseFloat(previousItem.order)) / 2 + nextOrder = (parseFloat(previousItem.order) + parseFloat(nextItem.order)) / 2 } await api.dbView.update(currentItem.id, { order: !isNaN(Number(nextOrder)) ? nextOrder.toString() : oldIndex.toString() })