Display toolbox initialization only once

kitodo · Sep 12, 2023 · 8a451a1 · 8a451a1
1 parent ef2ec00
commit 8a451a1
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/Classes/Controller/ToolboxController.php b/Classes/Controller/ToolboxController.php
@@ -271,6 +271,8 @@ private function renderImageManipulationTool()
 
  $this->view->assign('imageManipulation', true);
  $this->view->assign('parentContainer', $parentContainer);
+ // activate it here as it is the most common used tool
+ $this->view->assign('activateToolbox', true);
  }
 
  /**

diff --git a/Resources/Private/Templates/Toolbox/Main.html b/Resources/Private/Templates/Toolbox/Main.html
@@ -189,14 +189,16 @@
  </f:if>
 </ul>
 
-<script>
- window.addEventListener('tx-dlf-documentLoaded', (e) => {
- // Avoid instantiating dlfToolbox multiple times;
- // TODO(client-side): Rethink how it's done
- if (!window.tx_dlf_toolbox) {
- window.tx_dlf_toolbox = new dlfToolbox(e.detail.docController);
- }
- });
-</script>
+<f:if condition="{activateToolbox}">
+ <f:then>
+ <script>
+ window.addEventListener('tx-dlf-documentLoaded', (e) => {
+ if (!window.tx_dlf_toolbox) {
+ window.tx_dlf_toolbox = new dlfToolbox(e.detail.docController);
+ }
+ });
+ </script>
+ </f:then>
+</f:if>
 
 </html>