Won't fix Html invalid syntax in "template_edit" (admincp)

[refael]

Admin template "template_edit"

<xen:label>{xen:raw $styleChooser} {xen:phrase template_name}:</xen:label>

The output is something like

<label for="templateTitle">
    <div class="topChoosers">
        <div class="Popup">
            <span class="muted">Style:</span> <a rel="Menu">Master Style</a>
            <div class="Menu">
                <div class="primaryContent menuHeader"><h3>Styles</h3></div>
                <ul class="secondaryContent blockLinksList">
                        <li><a href="admin.php?styles/master-style.0/templates#_account-css"><span class="_depth0">Master Style</span></a></li>
                        <li><a href="admin.php?styles/default-style.1/templates#_account-css"><span class="_depth1">Default Style</span></a></li>
                </ul>
            </div>
        </div>
    </div>
Template Name:</label>

<div> inside a <label> tag is incorrect.

Not sure if you consider such errors worth fixing since it's in the admin area.

 

[Mike]

In general, we don't aim for HTML validation for the sake of validation; we would generally make changes if causes problems (rendering differences, accessibility issues, etc). As a general example, there are things that weren't allowed in HTML4 but are specifically allowed in HTML5 (such as <a> tags wrapping <div>'s) and browsers rendered them as expected already.

Worth checking this case, though there may not be a simple fix anyway.