[wp-trac] [WordPress Trac] #52851: Update the `element-closest` polyfill

WordPress Trac noreply at wordpress.org
Mon Feb 6 20:41:59 UTC 2023


#52851: Update the `element-closest` polyfill
-------------------------------------+---------------------
 Reporter:  desrosj                  |       Owner:  (none)
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  6.2
Component:  External Libraries       |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  needs-testing has-patch  |     Focuses:
-------------------------------------+---------------------
Changes (by desrosj):

 * keywords:  needs-testing has-patch changes-requested => needs-testing
     has-patch


Comment:

 I've gone and refreshed the PR against the current state of `trunk`.

 Thinking about this more, I'm leaning towards just using the minified
 version of the package in both versions of the file created.

 It's not used in Core anymore and is only maintained as a courtesy and to
 prevent 404 errors if being loaded directly. Additionally,
 [https://wpdirectory.net/search/01GRM77K77416FYH3K2WRY06RA there's almost
 no real world usage of this polyfill according to a search of the plugin
 library].

 I've also included this change in the PR. If this seems like the right
 route, I'll get it committed for 6.2. Otherwise we should just
 `maybelater` this one as there's really very little benefit.

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


More information about the wp-trac mailing list