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

        Arran Mccullough created issue -
        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.
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment Case12921Example.zip [ 17269 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        Assignee Priority P2 [ 10011 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #41622 Thu Jul 10 13:14:51 MDT 2014 art.zambrano ICE-10112 fix to render tabindex attribute
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/autocompleteentry/AutoCompleteEntryRenderer.java
        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.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        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.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: