[wp-trac] [WordPress Trac] #56122: Explore allowing more layout-specific CSS properties in safe_style_css filter

WordPress Trac noreply at wordpress.org
Tue Sep 6 03:01:04 UTC 2022


#56122: Explore allowing more layout-specific CSS properties in safe_style_css
filter
-------------------------------------------------+-------------------------
 Reporter:  andrewserong                         |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  Formatting                           |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests changes-    |     Focuses:
  requested                                      |
-------------------------------------------------+-------------------------

Comment (by andrewserong):

 Update: I've updated the linked pull request (https://github.com/WordPress
 /wordpress-develop/pull/2928) to implement each of these CSS properties
 except for `display`, so that we can side-step the potential issues raised
 by exposing that property. For the single use case of outputting `display`
 in the Theme JSON class, in the Gutenberg repo, we currently handle allow-
 listing and output of `display` directly rather than running it through
 `safecss_filter_attr`.

 So, I believe the PR is ready now for a final review for 6.1. Thanks
 @ramonopoly and @peterwilsoncc for taking a look at that PR.

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


More information about the wp-trac mailing list