Fix for 113 (#116)

adobe · Oct 7, 2024 · 6dc1c3c · 6dc1c3c
1 parent 4c2d011
commit 6dc1c3c
Showing 1 changed file with 27 additions and 17 deletions.
diff --git a/src/plugins/blocks/blocks.js b/src/plugins/blocks/blocks.js
@@ -288,23 +288,8 @@ function loadBlock(context, event, container) {
   // Set block title & description in UI
   updateDetailsContainer(content, authoredBlockName, blockDescription);
 
-  const disableCopyButton = sectionLibraryMetadata.disablecopy
-    ?? defaultLibraryMetadata.disablecopy
-    ?? false;
-
-  const copyButton = container.querySelector('.content .copy-button');
-  copyButton.removeAttribute('disabled');
-  if (disableCopyButton) {
-    copyButton.setAttribute('disabled', 'true');
-  }
-
-  const hideDetailsView = sectionLibraryMetadata.hidedetailsview
-  ?? defaultLibraryMetadata.hidedetailsview
-  ?? context.hidedetailsview
-  ?? false;
-
-  const splitView = container.querySelector('.content sp-split-view');
-  splitView.primarySize = hideDetailsView ? '100%' : '75%';
+  handleCopyButton(container, sectionLibraryMetadata, defaultLibraryMetadata);
+  handleSplitView(container, sectionLibraryMetadata, defaultLibraryMetadata, context);
 
   const blockRenderer = content.querySelector('block-renderer');
 
@@ -358,6 +343,9 @@ function loadTemplate(context, event, container) {
   // Set template title & description in UI
   updateDetailsContainer(content, authoredTemplateName, templateDescription);
 
+  handleCopyButton(container, sectionLibraryMetadata, defaultLibraryMetadata);
+  handleSplitView(container, sectionLibraryMetadata, defaultLibraryMetadata, context);
+
   const blockRenderer = content.querySelector('block-renderer');
 
   // If the block element exists, load the block
@@ -387,6 +375,28 @@ function loadTemplate(context, event, container) {
   sampleRUM('library:blockviewed', { target: blockData.url });
 }
 
+function handleCopyButton(container, sectionLibraryMetadata, defaultLibraryMetadata) {
+  const disableCopyButton = sectionLibraryMetadata?.disablecopy
+    ?? defaultLibraryMetadata?.disablecopy
+    ?? false;
+
+  const copyButton = container.querySelector('.content .copy-button');
+  copyButton.removeAttribute('disabled');
+  if (disableCopyButton) {
+    copyButton.setAttribute('disabled', 'true');
+  }
+}
+
+function handleSplitView(container, sectionLibraryMetadata, defaultLibraryMetadata, context) {
+  const hideDetailsView = sectionLibraryMetadata?.hidedetailsview
+    ?? defaultLibraryMetadata?.hidedetailsview
+    ?? context?.hidedetailsview
+    ?? false;
+
+  const splitView = container.querySelector('.content sp-split-view');
+  splitView.primarySize = hideDetailsView ? '100%' : '75%';
+}
+
 /**
  * Called when a user tries to load the plugin
  * @param {HTMLElement} container The container to render the plugin in