ICEfaces
  1. ICEfaces
  2. ICE-10428

ace:autoCompleteEntry XHTML Strict Compliance Issue

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: EE-4.0.0.GA, 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces ace:autoCompleteEntry

      Description

      The ace:autoCompleteEntry component uses the following markup structure, which nests a DIV inside of a SPAN, which is technically invalid for xhtml:

      <div id="frm:autoCompleteAccessKey">
                 <span style="display:inline-block;">
                      <input accesskey="c" aria-autocomplete="list" autocomplete="off" class="ui-inputfield ui-widget ui-state-default ui-corner-all ui-state-optional" name="frm:autoCompleteAccessKey_input" onblur="" onfocus="ice.setFocus(this.id);" onmousedown="this.focus();" role="textbox" style="width: 150px;null" type="text" />

       **** <span style="position:relative;display:block;">
       **** <div class="ui-widget ui-widget-content ui-corner-all" id="frm:autoCompleteAccessKey_div"
                       style="display:none;z-index:500;">
                     </div>
                   </span>
                 </span>
                 <script type="text/javascript">
                 </script>
                 <span id="frm:autoCompleteAccessKey_fieldupdate">
                   <script type="text/javascript">
                   </script>
                 </span>
                 <div id="frm:autoCompleteAccessKey_update">
                   <span>
                     <script type="text/javascript">
                     </script>
                   </span>
                 </div>
               </div>


      This JIRA is to modify the markup to not nest a DIV inside a SPAN so improve XHTML compliance.

        Activity

        Hide
        Arturo Zambrano added a comment -

        r43910: modified markup to comply with XHTML.

        Changed divs for spans and adjusted for changes in ace:autoCompleteEntry, ace:comboBox, and ace:selectMenu.

        Show
        Arturo Zambrano added a comment - r43910: modified markup to comply with XHTML. Changed divs for spans and adjusted for changes in ace:autoCompleteEntry, ace:comboBox, and ace:selectMenu.
        Hide
        Arturo Zambrano added a comment -

        Attaching the output of TotalValidator for the AutoCompleteEntryOnly.xhtml page of QA's test app. The component markup doesn't show any XHTML errors.

        Show
        Arturo Zambrano added a comment - Attaching the output of TotalValidator for the AutoCompleteEntryOnly.xhtml page of QA's test app. The component markup doesn't show any XHTML errors.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces4 trunk 4 r43917. Tomcat 7, IE 11, FF 34, Chrome 39. Visual testing ace:autoCompleteEntry, ace:comboBox, and ace:selectMenu in showcase did not find any issues.

        Show
        Liana Munroe added a comment - Verified ICEfaces4 trunk 4 r43917. Tomcat 7, IE 11, FF 34, Chrome 39. Visual testing ace:autoCompleteEntry, ace:comboBox, and ace:selectMenu in showcase did not find any issues.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: