From 0389d1876c49beb3aebee19d491b6669037fe49a Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Thu, 8 Jun 2023 01:00:49 +0530
Subject: [PATCH] fix: make cell selectable and skip paste behaviour if not
 permitted

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 packages/nc-gui/components/smartsheet/Grid.vue      | 4 ++--
 packages/nc-gui/composables/useMultiSelect/index.ts | 7 +++++++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/packages/nc-gui/components/smartsheet/Grid.vue b/packages/nc-gui/components/smartsheet/Grid.vue
index 07b213120b..387d376efd 100644
--- a/packages/nc-gui/components/smartsheet/Grid.vue
+++ b/packages/nc-gui/components/smartsheet/Grid.vue
@@ -954,9 +954,9 @@ const deleteSelectedRangeOfRows = () => {
                   <SmartsheetTableDataCell
                     v-for="(columnObj, colIndex) of fields"
                     :key="columnObj.id"
-                    class="cell relative cursor-pointer nc-grid-cell"
+                    class="cell relative nc-grid-cell"
                     :class="{
-                      'active': hasEditPermission && isCellSelected(rowIndex, colIndex),
+                      'active': isCellSelected(rowIndex, colIndex),
                       'nc-required-cell': isColumnRequiredAndNull(columnObj, row.row),
                       'align-middle': !rowHeight || rowHeight === 1,
                       'align-top': rowHeight && rowHeight !== 1,
diff --git a/packages/nc-gui/composables/useMultiSelect/index.ts b/packages/nc-gui/composables/useMultiSelect/index.ts
index 6c3e5390e6..9e7ec36d64 100644
--- a/packages/nc-gui/composables/useMultiSelect/index.ts
+++ b/packages/nc-gui/composables/useMultiSelect/index.ts
@@ -24,6 +24,7 @@ import {
   useI18n,
   useMetas,
   useProject,
+  useUIPermission,
 } from '#imports'
 
 const MAIN_MOUSE_PRESSED = 0
@@ -73,6 +74,9 @@ export function useMultiSelect(
     () => !(activeCell.row === null || activeCell.col === null || isNaN(activeCell.row) || isNaN(activeCell.col)),
   )
 
+  const { isUIAllowed } = useUIPermission()
+  const hasEditPermission = $computed(() => isUIAllowed('xcDatatableEditable'))
+
   function makeActive(row: number, col: number) {
     if (activeCell.row === row && activeCell.col === col) {
       return
@@ -343,6 +347,9 @@ export function useMultiSelect(
               // paste - ctrl/cmd + v
               case 86:
                 try {
+                  // if edit permission is not there, return
+                  if (!hasEditPermission) return
+
                   // handle belongs to column
                   if (
                     columnObj.uidt === UITypes.LinkToAnotherRecord &&