From 20a10cb8b715227a4f9d907e6ec7e2a366790d06 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Thu, 21 Oct 2021 13:50:08 +0530
Subject: [PATCH] fix: hide dropdown styling in public shared grid view

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 .../project/spreadsheet/components/editableCell.vue          | 4 ++--
 .../components/project/spreadsheet/views/xcGridView.vue      | 5 ++++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/packages/nc-gui/components/project/spreadsheet/components/editableCell.vue b/packages/nc-gui/components/project/spreadsheet/components/editableCell.vue
index fd5f8b758e..8d39585d7e 100644
--- a/packages/nc-gui/components/project/spreadsheet/components/editableCell.vue
+++ b/packages/nc-gui/components/project/spreadsheet/components/editableCell.vue
@@ -59,7 +59,7 @@
     />
 
     <enum-cell
-      v-else-if="isEnum && !isForm && !active"
+      v-else-if="isEnum && (( !isForm && !active) || isLocked || (isPublic && !isForm))"
       v-model="localState"
       :column="column"
       v-on="parentListeners"
@@ -81,7 +81,7 @@
     />
 
     <set-list-editable-cell
-      v-else-if="isSet && (active || isForm)"
+      v-else-if="isSet && (active || isForm) && !isLocked && !(isPublic && !isForm)"
       v-model="localState"
       :column="column"
       v-on="parentListeners"
diff --git a/packages/nc-gui/components/project/spreadsheet/views/xcGridView.vue b/packages/nc-gui/components/project/spreadsheet/views/xcGridView.vue
index 1a7d56cdc7..e3f147d46b 100644
--- a/packages/nc-gui/components/project/spreadsheet/views/xcGridView.vue
+++ b/packages/nc-gui/components/project/spreadsheet/views/xcGridView.vue
@@ -346,6 +346,8 @@ export default {
   },
   methods: {
     isRequired(_columnObj, rowObj) {
+      if (this.isPublicView) { return false }
+
       let columnObj = _columnObj
       if (columnObj.bt) {
         columnObj = this.meta.columns.find(c => c.cn === columnObj.bt.cn)
@@ -504,13 +506,14 @@ export default {
       }
     },
     enableEditable(column) {
-      return (column && column.uidt === 'Attachment') ||
+      return ((column && column.uidt === 'Attachment') ||
         (column && column.uidt === 'SingleSelect') ||
         (column && column.uidt === 'MultiSelect') ||
         (column && column.uidt === 'DateTime') ||
         (column && column.uidt === 'Date') ||
         (column && column.uidt === 'Time') ||
         (this.sqlUi && this.sqlUi.getAbstractType(column) === 'boolean')
+      )
     },
     insertNewRow(atEnd = false, expand = false) {
       this.$emit('insertNewRow', atEnd, expand)