[wp-trac] [WordPress Trac] #37968: HTML validation errors in admin tool

[WordPress Trac]

#37968: HTML validation errors in admin tool
----------------------------+------------------------------
 Reporter:  mdgl            |       Owner:  SergeyBiryukov
     Type:  defect (bug)    |      Status:  reviewing
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Administration  |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+------------------------------
Changes (by mdgl):

 * keywords:  has-patch reporter-feedback => has-patch


Comment:

 Sorry for the delay but I thought we were giving up on this ticket. A
 description of the errors appearing in the Microsoft Edge console log is
 included below.

 There don't appear to be any serious consequences of these errors, perhaps
 just the "embarrassment factor" because we are generating invalid HTML! It
 is possible though, that other browsers will behave differently when
 parsing the invalid markup and this might result in different styling
 being applied to various elements, depending on how they are interpreted
 by the browser. HTML5 does define some rules for how browsers are supposed
 to behave when presented with invalid markup, so this may not be a major
 issue, but I haven't had time to check.

 = Media -> Library =

 When displaying the media library screen in list mode, the following error
 appears because of an unclosed `<span>` element.

 {{{
 HTML1508: Unmatched end tag.
 upload.php (549,138)
 }}}

 = Settings -> General =

 The following errors appear on the general settings screen because of
 improperly nested `<span>` and `<input>` elements (in particular the
 `<span>` is opened inside the `<input>` element but closed outside).

 {{{
 HTML1508: Unmatched end tag.
 options-general.php (829,266)
 HTML1512: Unmatched end tag.
 options-general.php (829,469)
 HTML1508: Unmatched end tag.
 options-general.php (840,248)
 HTML1512: Unmatched end tag.
 options-general.php (840,446)
 }}}

 = Settings -> Akismet =

 There are two issues with the Akismet settings screen. Firstly, Akismet
 attempts to self-close a `<td>` element which (perhaps surprisingly) is
 not allowed in HTML5. Secondly, there is an unclosed `<span>` element.

 {{{
 HTML1500: Tag cannot be self-closing. Use an explicit closing tag.
 options-general.php (250,13)
 HTML1508: Unmatched end tag.
 options-general.php (273,184)
 HTML1500: Tag cannot be self-closing. Use an explicit closing tag.
 options-general.php (303,12)
 HTML1500: Tag cannot be self-closing. Use an explicit closing tag.
 options-general.php (310,12)
 }}}

 = Appearance -> Menus =

 Errors similar to the following appear if you use the appearance menus
 screen to display a menu that contains nested/hierarchical entries. The
 exact number of errors and their line numbers will depend on the menu
 structure. These errors occur because the `<li>` elements are not nested
 properly (see comment:2 above).

 {{{
 HTML1509: Unmatched end tag.
 nav-menus.php (1867,1)
 HTML1509: Unmatched end tag.
 nav-menus.php (1868,1)
 HTML1509: Unmatched end tag.
 nav-menus.php (2108,1)
 HTML1509: Unmatched end tag.
 nav-menus.php (2109,1)
 }}}

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