Merge pull request #638 from sewmyheadon/patch-3
Remove duplicate theme JS enqueue, update comment - Thx @sewmyheadon ! Argh, yes. You are right. We add the new version of adding it with a timestamp at the end to prevend caching issues while development...but it seems I forgot to remove the old request... Thx for fixing this!
This commit is contained in:
commit
725307dd6e
|
@ -7,7 +7,7 @@
|
|||
|
||||
if ( ! function_exists( 'understrap_scripts' ) ) {
|
||||
/**
|
||||
* Load theme's JavaScript sources.
|
||||
* Load theme's JavaScript and CSS sources.
|
||||
*/
|
||||
function understrap_scripts() {
|
||||
// Get the theme data.
|
||||
|
@ -22,7 +22,6 @@ if ( ! function_exists( 'understrap_scripts' ) ) {
|
|||
|
||||
$js_version = $theme_version . '.' . filemtime(get_template_directory() . '/js/theme.min.js');
|
||||
wp_enqueue_script( 'understrap-scripts', get_template_directory_uri() . '/js/theme.min.js', array(), $js_version, true );
|
||||
wp_enqueue_script( 'understrap-scripts', get_template_directory_uri() . '/js/theme.min.js', array(), $the_theme->get( 'Version' ), true );
|
||||
if ( is_singular() && comments_open() && get_option( 'thread_comments' ) ) {
|
||||
wp_enqueue_script( 'comment-reply' );
|
||||
}
|
||||
|
|
Reference in New Issue