[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