Changeset 2508

Timestamp:

02/15/2016 10:38:48 PM (9 years ago)

Author:

coffee2code

Message:

developer.wordpress.org: Handle migration of menu-ordered handbook page navigation functionality to handbook plugin.

• Remove wporg_developer_post_nav_via_menu().
• Use WPorg_Handbook_Navigation::navigate_via_menu() instead.
• Adjust related styling references.

See #425.

Location:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer

Files:

• inc/template-tags.php (modified) (1 diff)
• scss/main.scss (modified) (1 diff)
• single-plugin-handbook.php (modified) (1 diff)
• single-theme-handbook.php (modified) (1 diff)
• stylesheets/main.css (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/template-tags.php

@@ -65 +65 @@
                     <?php previous_post_link( '%link', _x( '<span class="meta-nav">&larr;</span> %title', 'Previous post link', 'wporg' ) ); ?>
                     <?php next_post_link( '%link', _x( '%title <span class="meta-nav">&rarr;</span>', 'Next post link', 'wporg' ) ); ?>
-
-                </div>
-                <!-- .nav-links -->
-            </nav><!-- .navigation -->
-        <?php
-        }
-    endif;
-
-    if ( ! function_exists( 'wporg_developer_post_nav_via_menu' ) ) :
-        /**
-         * Outputs previous and/or next post navigation links using the
-         * specified menu to inform navigation ordering.
-         *
-         * @param  string $menu_name The name of the menu to use for nav ordering.
-         */
-        function wporg_developer_post_nav_via_menu( $menu_name ) {
-            // Get the items for the specified menu
-            if ( ! $menu_items = wp_get_nav_menu_items( $menu_name ) ) {
-                return;
-            }
-
-            // Get ids for all menu objects
-            $menu_ids = wp_list_pluck( $menu_items, 'object_id' );
-
-            // Get current post
-            if ( ! $post = get_post() ) {
-                return;
-            }
-
-            // Index of current post in menu. Return if not in menu.
-            $i = array_search( $post->ID, $menu_ids );
-            if ( false === $i ) {
-                return;
-            }
-
-            // Find the previous post (note: preview menu item may not be a post)
-            $previous = null;
-            for ( $n = $i-1; $n >= 0; $n-- ) {
-                if ( isset( $menu_items[ $n ] ) && is_a( $menu_items[ $n ], 'WP_Post' ) ) {
-                    $previous = $menu_items[ $n ];
-                    break;
-                }
-            }
-
-            // Find the next post (note: next menu item may not be a post)
-            $next = null;
-            $max = count( $menu_items );
-            for ( $n = $i+1; $n < $max; $n++ ) {
-                if ( isset( $menu_items[ $n ] ) && is_a( $menu_items[ $n ], 'WP_Post' ) ) {
-                    $next = $menu_items[ $n ];
-                    break;
-                }
-            }
-
-            if ( ! $previous && ! $next ) {
-                return;
-            }
-            ?>
-
-            <nav class="navigation post-navigation" role="navigation">
-                <h1 class="screen-reader-text"><?php _e( 'Post navigation', 'wporg' ); ?></h1>
-                <div class="nav-links">
-
-                <?php
-                if ( $previous ) {
-                    printf( '<a href="%s" rel="previous"><span class="meta-nav">&larr;</span> %s</a>',
-                        esc_url( $previous->url ),
-                        esc_html( $previous->title )
-                    );
-                }
-
-                if ( $next ) {
-                    printf( '<a href="%s" rel="next">%s <span class="meta-nav">&rarr;</span></a>',
-                        esc_url( $next->url ),
-                        esc_html( $next->title )
-                    );
-                }
-                ?>
 
                 </div>

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/scss/main.scss

@@ -1329 +1329 @@
     }
 
-    nav.post-navigation .nav-links a {
+    nav.handbook-navigation .nav-links a {
         width: 49%;
         display: inline-block;

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/single-plugin-handbook.php

@@ -18 +18 @@
             <?php get_template_part( 'content', 'handbook'); ?>
 
-            <?php wporg_developer_post_nav_via_menu( 'Plugin Table of Contents' ); ?>
+            <?php \WPorg_Handbook_Navigation::navigate_via_menu( 'Plugin Table of Contents' ); ?>
 
         <?php endwhile; // end of the loop. ?>

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/single-theme-handbook.php

@@ -18 +18 @@
             <?php get_template_part( 'content', 'handbook'); ?>
 
-            <?php wporg_developer_post_nav_via_menu( 'Theme Table of Contents' ); ?>
+            <?php \WPorg_Handbook_Navigation::navigate_via_menu( 'Theme Table of Contents' ); ?>
 
         <?php endwhile; // end of the loop. ?>

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/stylesheets/main.css

@@ -1438 +1438 @@
   width: 100%;
 }
-.devhub-wrap nav.post-navigation .nav-links a {
+.devhub-wrap nav.handbook-navigation .nav-links a {
   width: 49%;
   display: inline-block;
 }
-.devhub-wrap nav.post-navigation .nav-links a[rel="prev"] {
+.devhub-wrap nav.handbook-navigation .nav-links a[rel="prev"] {
   text-align: left;
 }
-.devhub-wrap nav.post-navigation .nav-links a[rel="next"] {
+.devhub-wrap nav.handbook-navigation .nav-links a[rel="next"] {
   text-align: right;
   float: right;