[wp-trac] [WordPress Trac] #51812: Update jQuery step three
WordPress Trac
noreply at wordpress.org
Wed Jan 20 07:55:17 UTC 2021
#51812: Update jQuery step three
-------------------------------------------------+-------------------------
Reporter: azaozz | Owner:
| SergeyBiryukov
Type: task (blessed) | Status: reviewing
Priority: normal | Milestone: 5.7
Component: External Libraries | Version:
Severity: normal | Resolution:
Keywords: has-patch early needs-testing | Focuses: javascript
needs-dev-note |
-------------------------------------------------+-------------------------
Comment (by SergeyBiryukov):
Replying to [comment:21 Clorith]:
> Yes, PR 812 is good to go as a first push.
Thanks for the PR! Just noting it needs a refresh after [49944] / #46872
and [49973] / #52073.
The first three changes are fairly easy to resolve (just remove
`#doaction2`), but I'm less sure about the change in
`js/_enqueues/wp/widgets/text.js`. It looks like the change from this PR
takes a different approach at fixing the same issue, so we need to decide
on the preferred approach. Since [49973] appears to be tested and
confirmed, perhaps we no longer need `$( window ).on( 'load', ... )`
there? /cc @azaozz
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51812#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list