Fix: #5044 - refactor the loading indicator for dynamically loaded content

1 files changed, 3 insertions, 14 deletions

diff --git a/app/Module/ChartsBlockModule.php b/app/Module/ChartsBlockModule.php
index 4f00d7a743..fa123f7880 100644
--- a/app/Module/ChartsBlockModule.php
+++ b/app/Module/ChartsBlockModule.php
@@ -154,9 +154,10 @@ class ChartsBlockModule extends AbstractModule implements ModuleBlockInterface
                     $module = $this->module_service->findByInterface(InteractiveTreeModule::class)->first();
 
                     if ($module instanceof InteractiveTreeModule) {
-                        $title  = I18N::translate('Interactive tree of %s', $individual->fullName());
-                        $tv     = new TreeView();
+                        $title = I18N::translate('Interactive tree of %s', $individual->fullName());
+                        $tv    = new TreeView();
                         [$html, $js] = $tv->drawViewport($individual, 2);
+                        $js      = 'document.addEventListener("DOMContentLoaded", function() {' . $js . '});';
                         $content = $html . '<script>' . $js . '</script>';
                     } else {
                         $title   = I18N::translate('Interactive tree');
@@ -183,18 +184,6 @@ class ChartsBlockModule extends AbstractModule implements ModuleBlockInterface
     }
 
     /**
-     * Should this block load asynchronously using AJAX?
-     *
-     * Simple blocks are faster in-line, more complex ones can be loaded later.
-     *
-     * @return bool
-     */
-    public function loadAjax(): bool
-    {
-        return true;
-    }
-
-    /**
      * Can this block be shown on the tree’s home page?
      *
      * @return bool