[wp-trac] [WordPress Trac] #57575: Editor: Introduce HTML Tag Processor

WordPress Trac noreply at wordpress.org
Mon Jan 30 20:22:53 UTC 2023


#57575: Editor: Introduce HTML Tag Processor
-------------------------------------------------+-------------------------
 Reporter:  azaozz                               |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  6.2
Component:  Editor                               |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests gutenberg-  |     Focuses:
  merge                                          |
-------------------------------------------------+-------------------------

Comment (by dmsnell):

 Pardon me for my ignorance, but why would we want to remove the safety
 these checks provide? Based on the existing codebase I was under the
 impression this practice is somewhat normative and important, having also
 seen cases where people unintentionally pulled in class files with
 `require` that were already there, and thus created crashes where they
 weren't needed.

 This is why I preferred wrapping each class in the check, but given that
 the linter wants to indent the entire file, I moved them to `wp-html.php`
 as a compromise.

 If a version in Gutenberg were to somehow load first I wouldn't see that
 as problematic. We aren't wanting to break the API in the future.

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


More information about the wp-trac mailing list