Changeset 11655

Timestamp:

03/09/2022 06:06:20 AM (3 years ago)

Author:

dd32

Message:

Plugin Directory: FAQ: Respect the scroll-padding-top when scrolling to opened FAQ entries.

See #6025.

Location:

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

Files:

• functions.php (modified) (2 diffs)
• js/section-faq.js (modified) (1 diff)

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

@@ -74 +74 @@
  */
 function scripts() {
-    wp_enqueue_style( 'wporg-style', get_theme_file_uri( '/css/style.css' ), [ 'dashicons', 'open-sans' ], '20220118' );
+    wp_enqueue_style( 'wporg-style', get_theme_file_uri( '/css/style.css' ), [ 'dashicons', 'open-sans' ], filemtime( __DIR__ . '/css/style.css' ) );
     wp_style_add_data( 'wporg-style', 'rtl', 'replace' );
 
@@ -87 +87 @@
     if ( is_singular( 'plugin' ) ) {
         wp_enqueue_script( 'wporg-plugins-popover', get_stylesheet_directory_uri() . '/js/popover.js', array( 'jquery' ), '20171002', true );
-        wp_enqueue_script( 'wporg-plugins-faq', get_stylesheet_directory_uri() . '/js/section-faq.js', array( 'jquery' ), '20200120', true );
+        wp_enqueue_script( 'wporg-plugins-faq', get_stylesheet_directory_uri() . '/js/section-faq.js', array( 'jquery' ), filemtime( __DIR__ . '/js/section-faq.js' ), true );
     }
 

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/js/section-faq.js

@@ -38 +38 @@
 
             if ( window.location.hash ) {
-                window.scrollTo( 0, $question.offset().top );
+                var scrollPaddingTop = parseInt( $('html').css('scroll-padding-top') || 0 );
+
+                window.scrollTo( 0, $question.offset().top - scrollPaddingTop );
             }
         } );