[wp-trac] [WordPress Trac] #51519: Remove use of jQuery ready()

WordPress Trac noreply at wordpress.org
Tue Jun 8 15:56:18 UTC 2021


#51519: Remove use of jQuery ready()
-------------------------------------------------+-------------------------
 Reporter:  wpnomad                              |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  5.8
Component:  General                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-      |     Focuses:  javascript
  refresh                                        |
-------------------------------------------------+-------------------------

Comment (by desrosj):

 @pramodjodhani Thanks for the work! I did push an update to your branch to
 resolve the conflict with the scripts file.

 It looks like there are still a few more instances that need replacing
 (`$(document).ready` without spaces):
 - https://github.com/pramodjodhani/wordpress-develop/blob/ticket/51519/src
 /wp-admin/includes/class-wp-internal-pointers.php#L143
 - https://github.com/pramodjodhani/wordpress-develop/blob/ticket/51519/src
 /wp-admin/includes/class-custom-image-header.php#L396

 There's also [https://github.com/pramodjodhani/wordpress-
 develop/blob/ticket/51519/src/js/_enqueues/admin/common.js#L1711 one
 inline documentation comment that references this old behavior].

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/51519#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list