From 22562e325cae89e69d41ccbe18257c18f24f45e3 Mon Sep 17 00:00:00 2001
From: yonkov <yonkov.atanas@gmail.com>
Date: Tue, 15 Nov 2022 15:38:24 +0200
Subject: [PATCH] Add option to hide namespace sidebar

---
 client/web/compose/src/views/Layout.vue       | 17 +++++++++++
 .../web/compose/src/views/Namespace/Edit.vue  | 29 ++++++++++++++++++-
 .../web/compose/src/views/Namespace/View.vue  |  2 ++
 lib/js/src/compose/types/namespace.ts         |  2 +-
 .../src/components/navigation/CSidebar.vue    | 26 ++++++++++++-----
 .../en/corteza-webapp-compose/namespace.yaml  |  6 +++-
 6 files changed, 71 insertions(+), 11 deletions(-)

diff --git a/client/web/compose/src/views/Layout.vue b/client/web/compose/src/views/Layout.vue
index c18fad564..cfdf41090 100644
--- a/client/web/compose/src/views/Layout.vue
+++ b/client/web/compose/src/views/Layout.vue
@@ -178,14 +178,31 @@ export default {
      */
     this.$root.$on('alert', ({ message, ...params }) => this.toast(message, params))
     this.$root.$on('reminder.show', this.showReminder)
+
+    this.$root.$on('check-namespace-sidebar', this.checkNamespaceSidebar)
   },
 
   beforeDestroy () {
     this.$root.$off('alert')
     this.$root.$off('reminder.show', this.showReminder)
+    this.$root.$off('check-namespace-sidebar', this.checkNamespaceSidebar)
   },
 
   methods: {
+    checkNamespaceSidebar (showSidebar) {
+      const defaultDisabledRoutes = [
+        'namespaces',
+        'namespace.list',
+        'namespace.edit',
+        'namespace.create',
+        'namespace.clone',
+        'namespace.manage',
+      ]
+      const namespaceRoutes = ['page', 'pages', 'page.record', 'page.record.create', 'page.record.edit']
+
+      this.disabledRoutes = [...defaultDisabledRoutes, ...(showSidebar ? [] : namespaceRoutes)]
+    },
+
     removeToast (reminderID) {
       const i = this.toasts.findIndex(r => r.reminderID === reminderID)
       if (i > -1) {
diff --git a/client/web/compose/src/views/Namespace/Edit.vue b/client/web/compose/src/views/Namespace/Edit.vue
index 20c0ed26d..b86ea58cf 100644
--- a/client/web/compose/src/views/Namespace/Edit.vue
+++ b/client/web/compose/src/views/Namespace/Edit.vue
@@ -22,6 +22,19 @@
         >
           {{ $t('visit') }}
         </b-button>
+        <b-button
+          v-if="namespace.canManageNamespace"
+          :title="$t('configure')"
+          data-test-id="button-visit-admin-panel"
+          variant="primary"
+          class="d-flex align-items-center"
+          :to="{ name: 'admin.modules', params: { slug: namespace.slug } }"
+          style="margin-left:2px;"
+        >
+          <font-awesome-icon
+            :icon="['fas', 'cogs']"
+          />
+        </b-button>
         <namespace-translator
           v-if="namespace"
           :namespace="namespace"
@@ -213,7 +226,7 @@
 
             <b-form-group
               :label="$t('description.label')"
-              class="mb-0"
+              class="mb-3"
             >
               <b-input-group>
                 <b-form-textarea
@@ -232,6 +245,18 @@
                 </b-input-group-append>
               </b-input-group>
             </b-form-group>
+            <hr>
+
+            <b-form-group
+              :label="$t('sidebar.configure')"
+            >
+              <b-form-checkbox
+                v-model="namespace.meta.hideSidebar"
+                data-test-id="checkbox-show-sidebar"
+              >
+                {{ $t('sidebar.hide') }}
+              </b-form-checkbox>
+            </b-form-group>
           </b-form>
 
           <template
@@ -432,6 +457,7 @@ export default {
       this.namespace.meta = {
         subtitle: '',
         description: '',
+        hideSidebar: false,
         ...this.namespace.meta,
       }
 
@@ -550,6 +576,7 @@ export default {
       this.namespace.meta = {
         subtitle: '',
         description: '',
+        hideSidebar: false,
         ...this.namespace.meta,
       }
     },
diff --git a/client/web/compose/src/views/Namespace/View.vue b/client/web/compose/src/views/Namespace/View.vue
index c71a732c8..fb33c9a67 100644
--- a/client/web/compose/src/views/Namespace/View.vue
+++ b/client/web/compose/src/views/Namespace/View.vue
@@ -137,6 +137,8 @@ export default {
       this.$store.dispatch('chart/clearSet')
       this.$store.dispatch('page/clearSet')
 
+      this.$root.$emit('check-namespace-sidebar', !this.namespace.meta.hideSidebar)
+
       // Preload all data we need.
       Promise.all([
         this.$store.dispatch('module/load', p)
diff --git a/lib/js/src/compose/types/namespace.ts b/lib/js/src/compose/types/namespace.ts
index b88fa58bf..0f5d4dc05 100644
--- a/lib/js/src/compose/types/namespace.ts
+++ b/lib/js/src/compose/types/namespace.ts
@@ -12,7 +12,7 @@ interface MetaAdmin {
 interface Meta {
   subtitle: string;
   description: string;
-
+  hideSidebar: boolean;
   // Temporary icon & logo URLs
   // @todo rework this when we rework attachment management
   icon: string;
diff --git a/lib/vue/src/components/navigation/CSidebar.vue b/lib/vue/src/components/navigation/CSidebar.vue
index 68b91c44c..de1d8e045 100644
--- a/lib/vue/src/components/navigation/CSidebar.vue
+++ b/lib/vue/src/components/navigation/CSidebar.vue
@@ -226,19 +226,29 @@ export default {
   watch: {
     '$route.name': {
       immediate: true,
-      handler (name) {
-        // If sidebar should be disabled on route, close and unpin when navigating to route
-        if (this.disabledRoutes.includes(name)) {
-          this.isPinned = false
-          this.isExpanded = false
-        } else if(this.expandOnHover){
-          this.defaultSidebarAppearance()
-        }
+      handler () {
+        this.checkSidebar()
       },
     },
+
+    disabledRoutes: {
+      handler () {
+        this.checkSidebar()
+      },
+    }
   },
 
   methods: {
+    checkSidebar () {
+      // If sidebar should be disabled on route, close and unpin when navigating to route
+      if (this.disabledRoutes.includes(this.$route.name)) {
+        this.isPinned = false
+        this.isExpanded = false
+      } else if(this.expandOnHover){
+        this.defaultSidebarAppearance()
+      }
+    },
+
     onHover: throttle(function (expand) {
       if (!this.pinned && this.expandOnHover) {
         setTimeout(() => {
diff --git a/locale/en/corteza-webapp-compose/namespace.yaml b/locale/en/corteza-webapp-compose/namespace.yaml
index f96014094..4dd7c72c1 100644
--- a/locale/en/corteza-webapp-compose/namespace.yaml
+++ b/locale/en/corteza-webapp-compose/namespace.yaml
@@ -51,7 +51,7 @@ title: Low Code Namespaces
 tooltip:
   translations: Namespace translations
 visit: Visit namespace
-
+configure: Configure namespace
 
 manage:
   title: Manage namespaces
@@ -69,3 +69,7 @@ manage:
       name: Name
       handle: Handle
       last-change: Last change
+
+sidebar:
+  configure: Namespace pages
+  hide: Hide sidebar on namespace public pages