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

              Descending order - Click to sort in ascending order
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Hide
              Permalink
              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.
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #40871 Thu Apr 24 10:15:55 MDT 2014 art.zambrano ICE-9624 added logic to set submitted value to empty string when inField label is submitted, so the required validator can be triggered
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/maskedentry/MaskedEntryRenderer.java
              Hide
              Permalink
              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.
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #40852 Mon Apr 21 17:38:40 MDT 2014 art.zambrano ICE-9624 added 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
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/maskedentry/maskedentry.js
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Hide
              Permalink
              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
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #40831 Thu Apr 17 13:09:14 MDT 2014 art.zambrano ICE-9624 added code to set inField label after initializing component
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/maskedentry/maskedentry.js
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Assignee Priority P3 [ 10012 ]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Arturo Zambrano made changes -
              Field Original Value New Value
              Assignee Arturo Zambrano [ artzambrano ]
              Arturo Zambrano created issue -

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: