Issue #3393105: Added JavaScript alter hook to move to <head>.

7231a371 · ambient.impact · d14779ef · 7231a371
Commit 7231a371 authored 1 year ago by ambient.impact
--- a/modules/refreshless_turbo/src/Hooks/Javascript.php
+++ b/modules/refreshless_turbo/src/Hooks/Javascript.php
+<?php
+
+declare(strict_types=1);
+
+namespace Drupal\refreshless_turbo\Hooks;
+
+use Drupal\Core\Asset\AttachedAssetsInterface;
+use Drupal\Core\Language\LanguageInterface;
+use Drupal\hux\Attribute\Alter;
+
+/**
+ * JavaScript hook implementations.
+ */
+class Javascript {
+
+  #[Alter('js')]
+  /**
+   * Move all JavaScript to the <head> as recommended for Hotwire Turbo.
+   *
+   * JavaScript found in the <body> will be executed each time that page is
+   * loaded whereas JavaScript linked in the <head> will only be executed on a
+   * full page load. Since all Drupal JavaScript should be using behaviours,
+   * they should all correctly re-attach on every Turbo visit.
+   *
+   * @see \hook_js_alter()
+   */
+  public function alter(
+    array &$javascript,
+    AttachedAssetsInterface $assets, LanguageInterface $language,
+  ): void {
+
+    foreach ($javascript as $filePath => &$settings) {
+
+      // Skip items that are already set to the header as they may need to be
+      // render blocking rather than deferred.
+      if ($settings['scope'] === 'header') {
+        continue;
+      }
+
+      $settings['scope'] = 'header';
+
+      if (!isset($settings['attributes']['defer'])) {
+        $settings['attributes']['defer'] = true;
+      }
+
+    }
+
+  }
+
+}