From 4fa85c17bffb0771f974bd00edb71f684ad3d5d0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C5=BEe=20Fortun?= <joze.fortun@gmail.com>
Date: Thu, 6 Apr 2023 14:21:45 +0200
Subject: [PATCH] Fix switching pages in builder

---
 .../src/components/Common/RecordToolbar.vue   |  4 +-
 .../compose/src/views/Admin/Pages/Builder.vue | 37 ++++++++-----------
 2 files changed, 18 insertions(+), 23 deletions(-)

diff --git a/client/web/compose/src/components/Common/RecordToolbar.vue b/client/web/compose/src/components/Common/RecordToolbar.vue
index e9fe70116..aa8c746f3 100644
--- a/client/web/compose/src/components/Common/RecordToolbar.vue
+++ b/client/web/compose/src/components/Common/RecordToolbar.vue
@@ -120,8 +120,6 @@
             {{ labels.clone || $t('label.saveAsCopy') }}
           </b-button>
 
-          <slot name="end-actions" />
-
           <b-button
             v-if="!inEditing && isCreated && !(hideEdit || settings.hideEdit)"
             data-test-id="button-edit"
@@ -168,6 +166,8 @@
               {{ labels.submit || $t('label.save') }}
             </span>
           </b-button>
+
+          <slot name="end-actions" />
         </template>
       </b-col>
     </b-row>
diff --git a/client/web/compose/src/views/Admin/Pages/Builder.vue b/client/web/compose/src/views/Admin/Pages/Builder.vue
index f910a176b..fe39f01f4 100644
--- a/client/web/compose/src/views/Admin/Pages/Builder.vue
+++ b/client/web/compose/src/views/Admin/Pages/Builder.vue
@@ -12,7 +12,7 @@
 
     <portal to="topbar-tools">
       <b-form-select
-        v-if="layout"
+        v-if="layout && layouts.length > 1"
         ref="layoutSelect"
         size="sm"
         :value="layout.pageLayoutID"
@@ -318,6 +318,8 @@ export default {
 
   data () {
     return {
+      title: '',
+
       processing: false,
 
       page: undefined,
@@ -350,13 +352,6 @@ export default {
       },
     },
 
-    title () {
-      let { title = '', handle } = this.page || {}
-      title = title || handle
-
-      return this.$t('label.pageBuilder') + ' - ' + (title ? `"${title}"` : this.$t('label.noHandle'))
-    },
-
     showEditor () {
       return this.editor && this.editor.index !== undefined
     },
@@ -433,21 +428,21 @@ export default {
     pageID: {
       immediate: true,
       handler (pageID) {
-        this.page = undefined
-        this.layout = undefined
-        this.layouts = []
         this.unsavedBlocks.clear()
+        this.layouts = []
+        this.layout = undefined
 
-        if (pageID) {
-          const { namespaceID, name } = this.namespace
-          this.findPageByID({ namespaceID, pageID, force: true }).then(page => {
-            document.title = [page.title, name, this.$t('general:label.app-name.private')].filter(v => v).join(' | ')
-            this.page = page.clone()
-            return this.fetchPageLayouts()
-          }).then(() => {
-            this.setLayout()
-          })
-        }
+        const { namespaceID, name } = this.namespace
+        this.findPageByID({ namespaceID, pageID, force: true }).then(page => {
+          let { title = '', handle } = page
+          title = title || handle
+          this.title = `${this.$t('label.pageBuilder')} - "${title}"`
+          document.title = [page.title, name, this.$t('general:label.app-name.private')].filter(v => v).join(' | ')
+          this.page = page.clone()
+          return this.fetchPageLayouts()
+        }).then(() => {
+          this.setLayout()
+        })
       },
     },
   },