Skip to content

ST-958 Fix - Deprecated wp_script_add_data() conditional notice on wp 6.9+ - #57

Merged
deepench merged 1 commit into
masterfrom
ST-958-accelerate-theme-deprecated-warnings-on-php-8-4-and-wordpress-7
Oct 1, 2026
Merged

deepench merged 1 commit into
masterfrom
ST-958-accelerate-theme-deprecated-warnings-on-php-8-4-and-wordpress-7

Conversation

@jenisha00

Copy link
Copy Markdown

No description provided.

Only add the 'conditional' script data for html5shiv on WP < 6.9,
since core dropped IE conditional comment output in 6.9 and the
argument is deprecated there.
@jenisha00 jenisha00 self-assigned this Aug 4, 2026
@jenisha00 jenisha00 added the bug label Aug 4, 2026

@deepench deepench left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@deepench
deepench merged commit 5574d7d into master Oct 1, 2026
@deepench
deepench deleted the ST-958-accelerate-theme-deprecated-warnings-on-php-8-4-and-wordpress-7 branch October 1, 2026 03:26
@rajatgautam755421

Copy link
Copy Markdown

@jenisha00 heads-up: with the condition skipped on WP 6.9+, the html5shiv enqueue has no IE gate any more, so html5shiv.js loads in <head> for every visitor on current WordPress — before, it loaded for nobody. Verified on the Pro equivalent (themegrill/accelerate-pro#6) on WP 7.1.2.

#86 fixes this the way Spacious and Radiate did (empty html5shiv handle: no notice, nothing printed, dependents still load). Since that's the only change here, could this one be closed in favour of #86?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants