[wp-trac] [WordPress Trac] #54922: Normalizing CSS also catches CSS IDs instead of only URLs

WordPress Trac noreply at wordpress.org
Tue Feb 8 23:26:23 UTC 2022


#54922: Normalizing CSS also catches CSS IDs instead of only URLs
-------------------------------------------------+-------------------------
 Reporter:  mahype                               |       Owner:  audrasjb
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.9.1
Component:  Script Loader                        |     Version:  5.9
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests has-        |     Focuses:
  testing-info commit                            |
-------------------------------------------------+-------------------------
Changes (by audrasjb):

 * keywords:  has-patch has-unit-tests needs-testing has-testing-info =>
     has-patch has-unit-tests has-testing-info commit
 * owner:  (none) => audrasjb
 * status:  new => assigned


Comment:

 I tested the proposed PR and it fixes the issue.

 Using:
 {{{
 echo _wp_normalize_relative_css_links( 'clip-path: url(#image1);',
 get_stylesheet_uri() );
 }}}

 Result before patch:
 `clip-path: url(/552/wordpress-13/wp-
 content/themes/twentytwenty/#image1);`

 Result after patch:
 `clip-path: url(#image1);`

 The patch was reviewed by @SergeyBiryukov. It passes the unit tests.
 Self assigning for `commit`.

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


More information about the wp-trac mailing list