Skip to content

Commit cd570bd

Browse files
committed
Make wp-a11y an unconditional admin dependency
1 parent 81c93ac commit cd570bd

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

admin/class-enqueue-admin.php

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,12 +83,7 @@ public static function maybe_enqueue_admin_and_editor_app_scripts() {
8383
// let extensions supply the correct ID (e.g. a Pro virtual-page ID).
8484
$post_id = apply_filters( 'edac_filter_admin_post_id', $post_id );
8585

86-
$admin_script_dependencies = [ 'jquery' ];
87-
if ( 'accessibility_checker_settings' === $page ) {
88-
$admin_script_dependencies[] = 'wp-a11y';
89-
}
90-
91-
wp_enqueue_script( 'edac', plugin_dir_url( EDAC_PLUGIN_FILE ) . 'build/admin.bundle.js', $admin_script_dependencies, EDAC_VERSION, false );
86+
wp_enqueue_script( 'edac', plugin_dir_url( EDAC_PLUGIN_FILE ) . 'build/admin.bundle.js', [ 'jquery', 'wp-a11y' ], EDAC_VERSION, false );
9287
wp_set_script_translations( 'edac', 'accessibility-checker', plugin_dir_path( EDAC_PLUGIN_FILE ) . 'languages' );
9388
wp_localize_script(
9489
'edac',

tests/phpunit/Admin/EnqueueAdminTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public function testEnqueueBaseScriptInAdminNonEditorPage() {
9090

9191
$this->assertTrue( wp_script_is( 'edac', 'enqueued' ) );
9292
$this->assertFalse( wp_script_is( 'edac-editor-app', 'enqueued' ) );
93-
$this->assertNotContains( 'wp-a11y', $wp_scripts->registered['edac']->deps );
93+
$this->assertContains( 'wp-a11y', $wp_scripts->registered['edac']->deps );
9494

9595
$localized_data = $wp_scripts->get_data( 'edac', 'data' );
9696
$this->assertIsString( $localized_data );

0 commit comments

Comments
 (0)