ICEfaces
  1. ICEfaces
  2. ICE-10870

ice:inputTextArea focus is lost/interrupted w/ dynamic include

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P08
    • Fix Version/s: EE-1.8.2.GA_P09
    • Component/s: Framework, ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 1.8, Firefox, IE11

      Description

      When a page, that has input fields, is dynamically included, focus is lost/interrupted when tabbing through the fields.

      In the provided test case it has a form which includes commandLinks that change a ui:include. The content that is included after a click of the links has a form that has an ice:inputTextarea and an ice:inputText.

      The form that includes the inputs has partialSubmit="true" set on it. The inputTextarea also uses require="true".

      Upon including this content page and tabbing out of the inputTextarea field without adding anything the required validation is thrown. Focus should be set on the inputText field but it is placed on the commandLink that includes the content.

      Looking at the ice.focus parameter it is set to this on the request sent when tabbing out of the text area field.

      This is reproducible on Firefox and IE11. It works fine in Chrome. It also works fine if you replace the ice:inputTextarea with an ice:inputText component.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case to reproduce this issue.

        Steps:

        • Load welcomeICEfaces.iface
        • Click the 'Content One' link
        • Focus the cursor in the inputTextarea.
        • Tab out of the field. The required message is shown and focus is set to the 'Content One' link.
        Show
        Arran Mccullough added a comment - Attached test case to reproduce this issue. Steps: Load welcomeICEfaces.iface Click the 'Content One' link Focus the cursor in the inputTextarea. Tab out of the field. The required message is shown and focus is set to the 'Content One' link.
        Hide
        Mircea Toma added a comment -

        Modified renderer to record text area's focus so that the focus retention will apply focus back on the submitting element.

        Show
        Mircea Toma added a comment - Modified renderer to record text area's focus so that the focus retention will apply focus back on the submitting element.
        Hide
        Liana Munroe added a comment - - edited

        Tested with ICEfaces 1.8 r 49384, Tomcat 8, FF 47, Chrome 53, IE 11, using the attached test case.
        The focus does not move to the correct field after a blur event. When following the steps listed above "Tab out of the field", the required message is shown and focus is now flipped back to the inputTextArea and does not remain in the 2nd field.
        This fix also affects the component-showcase Text Entry demo:
        If you type through the text entries with the correct input there seems to be no issue, however, once you have entered text into the last textEntry the focus remains there. If you attempt to go back to type into a previous field, the cursor is returned to the last textEntry.

        Show
        Liana Munroe added a comment - - edited Tested with ICEfaces 1.8 r 49384, Tomcat 8, FF 47, Chrome 53, IE 11, using the attached test case. The focus does not move to the correct field after a blur event. When following the steps listed above "Tab out of the field", the required message is shown and focus is now flipped back to the inputTextArea and does not remain in the 2nd field. This fix also affects the component-showcase Text Entry demo: If you type through the text entries with the correct input there seems to be no issue, however, once you have entered text into the last textEntry the focus remains there. If you attempt to go back to type into a previous field, the cursor is returned to the last textEntry.
        Hide
        Mircea Toma added a comment -

        Complete focus retention fix by clearing out the recorded focus on blur.

        Show
        Mircea Toma added a comment - Complete focus retention fix by clearing out the recorded focus on blur.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 1.8 trunk r49389, Tomcat 7, IE 11, 10, 9, 8, 7, FF 47, Chrome 53. Attached test case is verified fixed as well as the component-showcase Text Entry demo.

        Show
        Liana Munroe added a comment - Verified ICEfaces 1.8 trunk r49389, Tomcat 7, IE 11, 10, 9, 8, 7, FF 47, Chrome 53. Attached test case is verified fixed as well as the component-showcase Text Entry demo.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: