Changeset 57115 for trunk/src/wp-includes/html-api/class-wp-html-processor.php

Timestamp:

11/17/2023 06:11:31 AM (3 years ago)

Author:

Bernhard Reiter

Message:

HTML API: Add support for containers elements, including ARTICLE.

There are a handful of elements which behave similarly and are generically container elements. These are the following elements:

ADDRESS, ARTICLE, ASIDE, BLOCKQUOTE, CENTER, DETAILS, DIALOG, DIR,
DL, DIV, FIELDSET, FIGCAPTION, FIGURE, FOOTER, HEADER, HGROUP, MAIN,
MENU, NAV, SEARCH, SECTION, SUMMARY

This patch adds support to the HTML Processor for handling these elements. They do not require any additional logic in the rest of the class, and carry no specific semantic rules for parsing beyond what is listed in their group in the IN BODY section of the HTML5 specification.

Props dmsnell.
Fixes #59914.

File:

• trunk/src/wp-includes/html-api/class-wp-html-processor.php (modified) (3 diffs)

trunk/src/wp-includes/html-api/class-wp-html-processor.php

@@ -100 +100 @@
  * The following list specifies the HTML tags that _are_ supported:
  *
+ *  - Containers: ADDRESS, BLOCKQUOTE, DETAILS, DIALOG, DIV, FOOTER, HEADER, MAIN, MENU, SPAN, SUMMARY.
+ *  - Form elements: BUTTON, FIELDSET, SEARCH.
+ *  - Formatting elements: B, BIG, CODE, EM, FONT, I, SMALL, STRIKE, STRONG, TT, U.
+ *  - Heading elements: HGROUP.
  *  - Links: A.
- *  - The formatting elements: B, BIG, CODE, EM, FONT, I, SMALL, STRIKE, STRONG, TT, U.
- *  - Containers: DIV, FIGCAPTION, FIGURE, SPAN.
- *  - Form elements: BUTTON.
+ *  - Lists: DL.
+ *  - Media elements: FIGCAPTION, FIGURE, IMG.
  *  - Paragraph: P.
- *  - Void elements: IMG.
+ *  - Sectioning elements: ARTICLE, ASIDE, NAV, SECTION
+ *  - Deprecated elements: CENTER, DIR
  *
  * ### Supported markup
@@ -622 +626 @@
                          * > "main", "menu", "nav", "ol", "p", "search", "section", "summary", "ul"
                          */
+                        case '+ADDRESS':
+                        case '+ARTICLE':
+                        case '+ASIDE':
                         case '+BLOCKQUOTE':
+                        case '+CENTER':
+                        case '+DETAILS':
+                        case '+DIALOG':
+                        case '+DIR':
                         case '+DIV':
+                        case '+DL':
+                        case '+FIELDSET':
                         case '+FIGCAPTION':
                         case '+FIGURE':
+                        case '+FOOTER':
+                        case '+HEADER':
+                        case '+HGROUP':
+                        case '+MAIN':
+                        case '+MENU':
+                        case '+NAV':
                         case '+P':
+                        case '+SEARCH':
+                        case '+SECTION':
+                        case '+SUMMARY':
                                 if ( $this->state->stack_of_open_elements->has_p_in_button_scope() ) {
                                         $this->close_a_p_element();
@@ -640 +662 @@
                          * > "menu", "nav", "ol", "pre", "search", "section", "summary", "ul"
                          */
+                        case '-ADDRESS':
+                        case '-ARTICLE':
+                        case '-ASIDE':
                         case '-BLOCKQUOTE':
                         case '-BUTTON':
+                        case '-CENTER':
+                        case '-DETAILS':
+                        case '-DIALOG':
+                        case '-DIR':
                         case '-DIV':
+                        case '-DL':
+                        case '-FIELDSET':
                         case '-FIGCAPTION':
                         case '-FIGURE':
+                        case '-FOOTER':
+                        case '-HEADER':
+                        case '-HGROUP':
+                        case '-MAIN':
+                        case '-MENU':
+                        case '-NAV':
+                        case '-SEARCH':
+                        case '-SECTION':
+                        case '-SUMMARY':
                                 if ( ! $this->state->stack_of_open_elements->has_element_in_scope( $tag_name ) ) {
                                         // @TODO: Report parse error.