Changeset 7231 for trunk/src/includes/common/template.php

Timestamp:

02/17/2022 06:44:23 PM (4 years ago)

Author:

johnjamesjacoby

Message:

Theme Compat: improve "no-js" body class swapping.

This change introduces a new helper function (bbp_swap_no_js_body_class()) which is used to output the <script> tag that was previously inside of the single template part that needed it.

It also adds a prefix (bbp) to those "no-js" body classes, with the intention of avoiding conflicts with other plugins & themes that may also be using the no-js body class for their own purposes.

Props dd32, r-a-y.

In trunk for 2.7.0. Fixes #3445.

File:

• trunk/src/includes/common/template.php (modified) (2 diffs)

trunk/src/includes/common/template.php

@@ -71 +71 @@
 
 /**
- * Add our custom head action to wp_head
+ * Add our custom footer action to wp_footer
  *
  * @since 2.0.0 bbPress (r2464)
@@ -1096 +1096 @@
     // Filter & return
     return (array) apply_filters( 'bbp_body_class', $classes, $bbp_classes, $wp_classes, $custom_classes );
+}
+
+/**
+ * Output a small piece of JavaScript to replace the "bbp-no-js" body class
+ * with "bbp-js" to allow interactive & dynamic elements to work as intended.
+ *
+ * @since 2.6.10 bbPress (r7229)
+ */
+function bbp_swap_no_js_body_class() {
+    static $done = false;
+
+    // Bail if already done
+    if ( true === $done ) {
+        return;
+    }
+
+    // Mark as done
+    $done = true;
+
+?>
+
+<script type="text/javascript" id="bbp-swap-no-js-body-class">
+    document.body.className = document.body.className.replace( 'bbp-no-js', 'bbp-js' );
+</script>
+
+<?php
 }