From aa1bddecda3f4b1dd248ff825b6233f72df288a8 Mon Sep 17 00:00:00 2001
From: HugoRCD <hugo.richard@epitech.eu>
Date: Mon, 24 Feb 2025 09:34:05 +0100
Subject: [PATCH] enhancement: Shift + click to select multiples variables

---
 .../shelve/app/components/variable/Create.vue |  6 -----
 apps/shelve/app/components/variable/Item.vue  |  8 ++++--
 .../projects/[projectId]/index/variables.vue  | 27 ++++++++++++++-----
 3 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/apps/shelve/app/components/variable/Create.vue b/apps/shelve/app/components/variable/Create.vue
index 42cff517..8b5e2b80 100644
--- a/apps/shelve/app/components/variable/Create.vue
+++ b/apps/shelve/app/components/variable/Create.vue
@@ -9,12 +9,6 @@ const { createLoading, createVariables } = useVariablesService()
 
 const route = useRoute()
 const projectId = route.params.projectId as string
-/*const project = useProject(projectId)
-
-const { data: secrets, status, refresh, error } = await useFetch('/api/github/secrets', {
-  method: 'GET',
-  query: { repository: sanitizeGithubUrl(project.value?.repository) },
-})*/
 
 const {
   variablesInput,
diff --git a/apps/shelve/app/components/variable/Item.vue b/apps/shelve/app/components/variable/Item.vue
index 6f628340..c1cc7252 100644
--- a/apps/shelve/app/components/variable/Item.vue
+++ b/apps/shelve/app/components/variable/Item.vue
@@ -39,15 +39,19 @@ const variableToUpdate = computed(() => ({
 }))
 
 const showEdit = ref(false)
+
+const handleClick = (event: MouseEvent) => {
+  emit('toggleSelected', event)
+}
 </script>
 
 <template>
   <UCard variant="subtle" :ui="{ root: isSelected && !showEdit ? 'bg-(--ui-bg-accented)/60' : '' }">
-    <div class="flex w-full items-start justify-between">
+    <div class="flex w-full select-none items-start justify-between">
       <div
         class="flex w-full flex-col gap-1"
         :class="{ 'cursor-pointer': !showEdit }"
-        @click="showEdit ? null : emit('toggleSelected')"
+        @click="showEdit ? null : handleClick($event)"
       >
         <h3 class="flex items-center gap-1 text-sm font-semibold sm:text-base">
           <span class="lg:hidden">
diff --git a/apps/shelve/app/pages/[teamSlug]/projects/[projectId]/index/variables.vue b/apps/shelve/app/pages/[teamSlug]/projects/[projectId]/index/variables.vue
index 798f95ce..759a2ad2 100644
--- a/apps/shelve/app/pages/[teamSlug]/projects/[projectId]/index/variables.vue
+++ b/apps/shelve/app/pages/[teamSlug]/projects/[projectId]/index/variables.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { TeamRole, type Variable } from '@types'
+import type { Variable } from '@types'
 
 const route = useRoute()
 const projectId = route.params.projectId as string
@@ -10,6 +10,7 @@ const { loading, fetchVariables } = useVariablesService()
 if (!variables.value) fetchVariables()
 
 const selectedVariables = ref<Variable[]>([])
+const lastSelectedIndex = ref<number | null>(null)
 const searchTerm = ref('')
 const selectedEnvironment = ref([])
 const order = ref('desc')
@@ -50,11 +51,25 @@ const filteredVariables = computed(() => {
   return filtered
 })
 
-const toggleVariable = (variable: Variable) => {
-  if (!isVariableSelected(variable)) {
-    selectedVariables.value.push(variable)
+const toggleVariable = (variable: Variable, event?: MouseEvent) => {
+  const filteredIndex = filteredVariables.value.findIndex(v => v.id === variable.id)
+
+  if (event?.shiftKey && lastSelectedIndex.value !== null) {
+    const startIndex = Math.min(lastSelectedIndex.value, filteredIndex)
+    const endIndex = Math.max(lastSelectedIndex.value, filteredIndex)
+    const newSelection = filteredVariables.value.slice(startIndex, endIndex + 1)
+    selectedVariables.value = Array.from(
+      new Map(
+        [...selectedVariables.value, ...newSelection].map(item => [item.id, item])
+      ).values()
+    )
   } else {
-    selectedVariables.value = selectedVariables.value.filter((v) => v.id !== variable.id)
+    if (!isVariableSelected(variable)) {
+      selectedVariables.value.push(variable)
+    } else {
+      selectedVariables.value = selectedVariables.value.filter((v) => v.id !== variable.id)
+    }
+    lastSelectedIndex.value = filteredIndex
   }
 }
 
@@ -91,7 +106,7 @@ const isVariableSelected = (variable: Variable) => {
           :variable
           :environments
           :is-selected="isVariableSelected(variable)"
-          @toggle-selected="toggleVariable(variable)"
+          @toggle-selected="(e) => toggleVariable(variable, e)"
         />
       </div>
     </div>