ICEfaces
  1. ICEfaces
  2. ICE-9624

ace:maskedEntry, the inField label is not shown after initializing the component

    Details

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

      Description

      After initializing an ace:maskedEntry component that uses an inField label, the label doesn't not appear in the text field at first. It only appears after the text field has been focused and then blurred.

        Activity

        Hide
        Arturo Zambrano added a comment -

        Committed fix to 4.0 trunk at revision 40831 and to 3.3 EE maintenance branch at revision 40832. Added the code to set the inField label at the end of the initialization process as well.

        Show
        Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 40831 and to 3.3 EE maintenance branch at revision 40832. Added the code to set the inField label at the end of the initialization process as well.
        Hide
        Liana Munroe added a comment - - edited

        When the label is inField the "required" attribute no longer functions as expected on that field. When the field is required and submitted without input, the value of the label gets submitted and the red required warning border around the field does not appear. This can be seen in test application at:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/maskedEntry
        The application is called MaskedEntry Optional Indicator Test.
        Occurs on both Icefaces 4 and maintenance branch trunk. 40839

        Show
        Liana Munroe added a comment - - edited When the label is inField the "required" attribute no longer functions as expected on that field. When the field is required and submitted without input, the value of the label gets submitted and the red required warning border around the field does not appear. This can be seen in test application at: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/maskedEntry The application is called MaskedEntry Optional Indicator Test. Occurs on both Icefaces 4 and maintenance branch trunk. 40839
        Hide
        Arturo Zambrano added a comment -

        Committed fix to trunk at revision 40852 and to 3.3 EE maintenance branch at revision 40853. The issue was already present, it just became more evident. It was fixed by adding logic to dynamically change the 'name' attribute of the text field when using the inField label, in order to avoid submitting the label itself as the value of the component.

        Show
        Arturo Zambrano added a comment - Committed fix to trunk at revision 40852 and to 3.3 EE maintenance branch at revision 40853. The issue was already present, it just became more evident. It was fixed by adding logic to dynamically change the 'name' attribute of the text field when using the inField label, in order to avoid submitting the label itself as the value of the component.
        Hide
        Liana Munroe added a comment -

        This fix is confirmed as far as not submitting the label itself when the label is "inField" and the field is required. In our test application the red warning border that surrounds the text field is not seen when the label is inField and submitted with no data unless the label position has been changed dynamically from another position to inField. Also when the inField position is hard coded into the test application the red border does not appear. Please advise if this is the expected behavior.
        Test application:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/maskedEntry
        Go to MaskedEntry Optional Indicator test.
        Check Required.
        Change Label Position to inField.
        Press Submit.
        Red border on input field is not rendered.
        Change Label Position to left.
        Press Submit.
        Red border is now rendered.

        Show
        Liana Munroe added a comment - This fix is confirmed as far as not submitting the label itself when the label is "inField" and the field is required. In our test application the red warning border that surrounds the text field is not seen when the label is inField and submitted with no data unless the label position has been changed dynamically from another position to inField. Also when the inField position is hard coded into the test application the red border does not appear. Please advise if this is the expected behavior. Test application: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/maskedEntry Go to MaskedEntry Optional Indicator test. Check Required. Change Label Position to inField. Press Submit. Red border on input field is not rendered. Change Label Position to left. Press Submit. Red border is now rendered.
        Hide
        Arturo Zambrano added a comment -

        Committed fix for issue above to 4.0 trunk at revision 40871 and to 3.3 EE maintenance branch at revision 40872. Added logic to set submitted value to empty string when inField label gets submitted, so the required validator can be triggered.

        Show
        Arturo Zambrano added a comment - Committed fix for issue above to 4.0 trunk at revision 40871 and to 3.3 EE maintenance branch at revision 40872. Added logic to set submitted value to empty string when inField label gets submitted, so the required validator can be triggered.
        Hide
        Liana Munroe added a comment -

        Confirmed fixed Icefaces 4 trunk and Icefaces ee-3.3.0 maintenance branch r40876. Tomcat 7, all browsers.

        Show
        Liana Munroe added a comment - Confirmed fixed Icefaces 4 trunk and Icefaces ee-3.3.0 maintenance branch r40876. Tomcat 7, all browsers.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: