ICEfaces
  1. ICEfaces
  2. ICE-10112

ace:autoCompleteEntry - tabindex attribute is not functional

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA, EE-3.3.0.GA_P02
    • Fix Version/s: 4.0, EE-3.3.0.GA_P03
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      Setting the tabindex attribute for the ace:autoCompleteEntry does not set the tabindex for the rendered input element. It also causes tab focus order to be incorrect in a form.

      In a case where there are 5 input elements on a page as follows:
      Input One - tabindex = 0
      Input Two - tabindex = 1
      Auto Complete Input - tabindex = 2
      Input Three - tabindex = 3
      Input Four - tabindex = 4

      Tabbing from Input One, the Auto Complete Input received focus, then the browser url bar, tabbing further will eventually go to Input Two and then the others.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case source code and war.

        Steps:

        • Load welcomeICEfaces.jsf
        • Focus into the first input text
        • Tab out of this input, focus is now on the auto complete entry and not the input directly below.
        Show
        Arran Mccullough added a comment - Attached test case source code and war. Steps: Load welcomeICEfaces.jsf Focus into the first input text Tab out of this input, focus is now on the auto complete entry and not the input directly below.
        Hide
        Arturo Zambrano added a comment -

        Committed fix to render tabindex attribute on text input field at revision 41622.

        Note that tabindexes start from 1. A zero value simply means that it follows the order in which the element appears in the source.

        Show
        Arturo Zambrano added a comment - Committed fix to render tabindex attribute on text input field at revision 41622. Note that tabindexes start from 1. A zero value simply means that it follows the order in which the element appears in the source.
        Hide
        Liana Munroe added a comment - - edited

        Verified Icefaces 4 trunk r41625, Tomcat 7, all browsers. Used the attached test case, Tabbing from tabindex 1 through 4 is successful.
        Test applications for Tab Index attribute have been added to:
        C:\svn\repo\qa\trunk\Regression-Icefaces4\Sparkle\Nightly\autoCompleteEntry
        and
        C:\svn\repo\qa\trunk\Regression-Icefaces2\Sparkle\Nightly\autoCompleteEntry

        Show
        Liana Munroe added a comment - - edited Verified Icefaces 4 trunk r41625, Tomcat 7, all browsers. Used the attached test case, Tabbing from tabindex 1 through 4 is successful. Test applications for Tab Index attribute have been added to: C:\svn\repo\qa\trunk\Regression-Icefaces4\Sparkle\Nightly\autoCompleteEntry and C:\svn\repo\qa\trunk\Regression-Icefaces2\Sparkle\Nightly\autoCompleteEntry
        Hide
        Ken Fyten added a comment -

        Re-opened to backport this to the 3.3 maintenance branch.

        Show
        Ken Fyten added a comment - Re-opened to backport this to the 3.3 maintenance branch.
        Hide
        Arturo Zambrano added a comment -

        r41626: committed fix to 3.3 EE maintenance branch.

        Show
        Arturo Zambrano added a comment - r41626: committed fix to 3.3 EE maintenance branch.
        Hide
        Liana Munroe added a comment -

        Verified Icefaces EE 3.3.0 maintenance branch r41626. Tomcat 7, all browsers.

        Show
        Liana Munroe added a comment - Verified Icefaces EE 3.3.0 maintenance branch r41626. Tomcat 7, all browsers.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: