Changeset 12990

Timestamp:

11/30/2023 10:37:12 PM (13 months ago)

Author:

coffee2code

Message:

Support Theme: Ensure menu link is marked as active if the link is for the current page, regardless of how the link was originally defined.

Props hiteshtalpada, coffee2code.
Fixes #7104.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/header.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/header.php

@@ -51 +51 @@
                                 <?php
                                 foreach ( $menu_items as $path => $text ) :
-                                    $class = '';
                                     $url = parse_url( $path );
+
+                                    // Check both host and path (if available).
+                                    $is_same_host = ! empty( $url['host'] ) ? $url['host'] === $_SERVER['HTTP_HOST'] : true;
+                                    $is_same_path = ! empty( $url['path'] ) && false !== strpos( $_SERVER['REQUEST_URI'], $url['path'] );
+
+                                    $class = ( $is_same_host && $is_same_path ) ? 'class="active" ' : '';
+
                                     if ( ! empty( $url['host' ] ) ) {
                                         $url = esc_url( $path );
                                     } else {
-                                        $class = false !== strpos( $_SERVER['REQUEST_URI'], $url['path'] ) ? 'class="active" ' : '';
                                         $url = esc_url( home_url( $path ) );
                                     }