[wp-trac] [WordPress Trac] #55356: Use safecss_filter_attr() instead of wp_strip_all_tags() in wp_get_layout_style()

WordPress Trac noreply at wordpress.org
Sat Mar 12 00:27:39 UTC 2022


#55356: Use safecss_filter_attr() instead of wp_strip_all_tags() in
wp_get_layout_style()
------------------------------+----------------------------
 Reporter:  grapplerulrich    |       Owner:  davidbaumwald
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  6.0
Component:  Formatting        |     Version:  5.9
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |     Focuses:
------------------------------+----------------------------
Changes (by davidbaumwald):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 In [changeset:"52924" 52924]:
 {{{
 #!CommitTicketReference repository="" revision="52924"
 Formatting: Use `safecss_filter_attr` in `wp_get_layout_style()`.

 Previously, `safecss_filter_attr` did not allow for `calc` and `var` in
 parsed CSS.  Because of this, `wp_strip_all_tags` was used in
 `wp_get_layout_style` to allow for the use of these CSS functions to
 account for responsive widths.  [50923] added support for both of these
 CSS functions in KSES, so `safecss_filter_attr` can now be used in
 `wp_get_layout_style` as originally intended.

 Follow-up to [52069].

 Props grapplerulrich.
 Fixes #55356.
 }}}

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


More information about the wp-trac mailing list