feat(task): always insert new tasks at the top

Resolves https://community.vikunja.io/t/kanban-cards-in-wrong-order/2731

frontend/src/components/project/views/ProjectKanban.vue

@@ -564,15 +564,15 @@ async function addTaskToBucket(bucketId: IBucket['id']) {
 	})
 	newTaskText.value = ''
 	kanbanStore.addTaskToBucket(task)
-	scrollTaskContainerToBottom(bucketId)
+	scrollTaskContainerToTop(bucketId)
 }
 
-function scrollTaskContainerToBottom(bucketId: IBucket['id']) {
+function scrollTaskContainerToTop(bucketId: IBucket['id']) {
 	const bucketEl = taskContainerRefs.value[bucketId]
 	if (!bucketEl) {
 		return
 	}
-	bucketEl.scrollTop = bucketEl.scrollHeight
+	bucketEl.scrollTop = 0
 }
 
 async function createNewBucket() {

frontend/src/stores/kanban.ts

@@ -194,8 +194,8 @@ export const useKanbanStore = defineStore('kanban', () => {
 			...oldBucket,
 			count: (oldBucket?.count || 0) + 1,
 			tasks: [
-				...oldBucket.tasks,
 				task,
+				...oldBucket.tasks,
 			],
 		}
 		buckets.value[bucketIndex] = newBucket

pkg/models/task_position.go

@@ -192,3 +192,39 @@ func getPositionsForView(s *xorm.Session, view *ProjectView) (positions []*TaskP
 		Find(&positions)
 	return
 }
+
+func calculateNewPositionForTask(s *xorm.Session, a web.Auth, t *Task, view *ProjectView) (*TaskPosition, error) {
+	if t.Position == 0 {
+		lowestPosition := &TaskPosition{}
+		exists, err := s.Where("project_view_id = ?", view.ID).
+			OrderBy("position asc").
+			Get(lowestPosition)
+		if err != nil {
+			return nil, err
+		}
+		if exists {
+			if lowestPosition.Position == 0 {
+				err = RecalculateTaskPositions(s, view, a)
+				if err != nil {
+					return nil, err
+				}
+
+				lowestPosition = &TaskPosition{}
+				_, err = s.Where("project_view_id = ?", view.ID).
+					OrderBy("position asc").
+					Get(lowestPosition)
+				if err != nil {
+					return nil, err
+				}
+			}
+
+			t.Position = lowestPosition.Position / 2
+		}
+	}
+
+	return &TaskPosition{
+		TaskID:        t.ID,
+		ProjectViewID: view.ID,
+		Position:      calculateDefaultPosition(t.Index, t.Position),
+	}, nil
+}

pkg/models/tasks.go

@@ -791,11 +791,12 @@ func createTask(s *xorm.Session, t *Task, a web.Auth, updateAssignees bool, setB
 			})
 		}
 
-		positions = append(positions, &TaskPosition{
+		newPosition, err := calculateNewPositionForTask(s, a, t, view)
-			TaskID:        t.ID,
+		if err != nil {
-			ProjectViewID: view.ID,
+			return err
-			Position:      calculateDefaultPosition(t.Index, t.Position),
+		}
-		})
+
+		positions = append(positions, newPosition)
 	}
 
 	if len(positions) > 0 {
@@ -955,11 +956,11 @@ func (t *Task) Update(s *xorm.Session, a web.Auth) (err error) {
 				return err
 			}
 
-			tp := TaskPosition{
+			tp, err := calculateNewPositionForTask(s, a, t, view)
-				TaskID:        t.ID,
+			if err != nil {
-				ProjectViewID: view.ID,
+				return err
-				Position:      calculateDefaultPosition(t.Index, t.Position),
 			}
+
 			err = tp.Update(s, a)
 			if err != nil {
 				return err