From f833300ebc2c86ff8d89ef8a5971fd7556a3703b Mon Sep 17 00:00:00 2001
From: Malcolm Smith <malcolmsmith18@gmail.com>
Date: Fri, 19 Jun 2026 17:00:04 +0000
Subject: [PATCH] fix: address PR review UI and syntax feedback

---
 .../components/project/views/ProjectList.vue  |  3 +-
 .../components/project/views/ViewEditForm.vue | 29 ++++++++++---------
 .../tasks/partials/SingleTaskInProject.vue    | 19 ++++--------
 pkg/models/project_view.go                    |  1 -
 4 files changed, 22 insertions(+), 30 deletions(-)

diff --git a/frontend/src/components/project/views/ProjectList.vue b/frontend/src/components/project/views/ProjectList.vue
index c352731f4..51da0ce7d 100644
--- a/frontend/src/components/project/views/ProjectList.vue
+++ b/frontend/src/components/project/views/ProjectList.vue
@@ -204,8 +204,7 @@ onMounted(async () => {
 	ctaVisible.value = true
 })
 
-const canDragTasks = computed(() => (canWrite.value || isSavedFilter(project.value)) && !includeSubprojects.value)
-
+const canDragTasks = computed(() => canWrite.value || isSavedFilter(project.value))
 const isTouchDevice = ref(false)
 if (typeof window !== 'undefined') {
 	isTouchDevice.value = !window.matchMedia('(hover: hover) and (pointer: fine)').matches
diff --git a/frontend/src/components/project/views/ViewEditForm.vue b/frontend/src/components/project/views/ViewEditForm.vue
index a24a45f7d..ba44a28fc 100644
--- a/frontend/src/components/project/views/ViewEditForm.vue
+++ b/frontend/src/components/project/views/ViewEditForm.vue
@@ -190,21 +190,22 @@ function handleBubbleSave() {
 			<FilterInputDocs />
 		</div>
 
-<FancyCheckbox
-          v-model="view.includeSubprojects"
-          v-tooltip="$t('project.views.includeSubprojects')"
-          class="mbe-3"
-        >
-          {{ $t('project.views.includeSubprojects') }}
-        </FancyCheckbox>
+<div class="field mbe-3">
+			<FancyCheckbox
+				v-model="view.includeSubprojects"
+				v-tooltip="$t('project.views.includeSubprojects')"
+			>
+				{{ $t('project.views.includeSubprojects') }}
+			</FancyCheckbox>
+		</div>
 
-        <div class="field mbe-3">
-          <FancyCheckbox
-            v-model="view.filter.filterIncludeNulls"
-          >
-            {{ $t('filters.attributes.includeNulls') }}
-          </FancyCheckbox>
-        </div>
+		<div class="field mbe-3">
+			<FancyCheckbox
+				v-model="view.filter.filter_include_nulls"
+			>
+				{{ $t('filters.attributes.includeNulls') }}
+			</FancyCheckbox>
+		</div>
 
 		<div
 			v-if="view.viewKind === 'kanban'"
diff --git a/frontend/src/components/tasks/partials/SingleTaskInProject.vue b/frontend/src/components/tasks/partials/SingleTaskInProject.vue
index 992b1953d..4bad3086c 100644
--- a/frontend/src/components/tasks/partials/SingleTaskInProject.vue
+++ b/frontend/src/components/tasks/partials/SingleTaskInProject.vue
@@ -153,15 +153,14 @@
 			/>
 
 			<RouterLink
-				v-if="showProjectSeparately"
-				v-tooltip="$t('task.detail.belongsToProject', {project: project.title})"
-				:to="{ name: 'project.index', params: { projectId: task.projectId } }"
-				class="task-project"
-				@click.stop
+			v-if="showProjectSeparately && project"
+			v-tooltip="$t('task.detail.belongsToProject', {project: project.title})"
+			:to="{ name: 'project.index', params: { projectId: task.projectId } }"
+			class="task-project"
+			@click.stop
 			>
-				{{ projectLabel }}
+			{{ project.title }}
 			</RouterLink>
-
 			<BaseButton
 				:class="{'is-favorite': task.isFavorite}"
 				class="favorite"
@@ -292,13 +291,7 @@ const currentProject = computed(() => {
 	} : baseStore.currentProject
 })
 
-const projectLabel = computed(() => {
-	if (!project.value) {
-		return ''
-	}
 
-	return project.value.title
-})
 
 const taskDetailRoute = computed(() => ({
 	name: 'task.detail',
diff --git a/pkg/models/project_view.go b/pkg/models/project_view.go
index 85aa6fe6a..dbb795f03 100644
--- a/pkg/models/project_view.go
+++ b/pkg/models/project_view.go
@@ -162,7 +162,6 @@ type ProjectView struct {
 	// The filter query to match tasks by. Check out https://vikunja.io/docs/filters for a full explanation.
 	Filter *TaskCollection `xorm:"json null default null" query:"filter" json:"filter" doc:"The filter query used to match tasks shown in this view. See https://vikunja.io/docs/filters."`
 	// The position of this view in the list. The list of all views will be sorted by this position.
-// The position of this view in the list. The list of all views will be sorted by this position.
 	Position float64 `xorm:"double null" json:"position" doc:"The position of this view in the list. The list of all views will be sorted by this position."`
 	// If enabled, this view also includes tasks from all descendant subprojects.
 	IncludeSubprojects bool `xorm:"default false not null" json:"include_subprojects"`