Changeset 9631

Timestamp:

03/25/2020 06:40:50 AM (5 years ago)

Author:

coffee2code

Message:

Theme Directory: Sync the postsPerPage value localized to the JS with value set via posts_per_page option.

In preparation of reducing pageload overhead by loading 12 themes at a time instead of 24. This change will allow JS and non-JS values to match and be changeable via WP admin.

Props dufresnesteven.
See #5062.

Location:

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

Files:

• functions.php (modified) (1 diff)
• js/theme.js (modified) (1 diff)

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

@@ -135 +135 @@
                 ),
                 'isMobile'     => wp_is_mobile(),
-                'postsPerPage' => 24,
+                'postsPerPage' => get_option( 'posts_per_page' ),
                 'path'         => trailingslashit( parse_url( home_url(), PHP_URL_PATH ) ),
                 'locale'       => get_locale(),

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

@@ -234 +234 @@
             instance = instance || 0;
 
-            // Themes per instance are set at 24
+            // Themes per instance are set via posts_per_page option in WP admin.
             collection = _( collection.rest( themes.data.settings.postsPerPage * instance ) );
             collection = _( collection.first( themes.data.settings.postsPerPage ) );