ICEfaces
  1. ICEfaces
  2. ICE-11305

ace:maskedEntry requires improvement

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: EE-4.2.0.GA
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Tomcat 8, all browsers. These issues have existed since 4.0
    • Assignee Priority:
      P2

      Description

      These issues have been present since 4.0
      1.) The mask becomes the value when ajax is used in the maskedEntry and the field is submit without entering input. See screenshot 1.PNG. Can also be seen in public showcase maskedEntry demos.
      To reproduce:
       - Use the maskedEntry Overview demo at: http://icefaces-showcase.icesoft.org/showcase.jsf?grp=ace:maskedEntry
       - Place focus in each form field and press Enter, the masks become the values in the Entered data section.


      2.) Scenario: ajax is not used and there is an inField label in the maskedEntry. If the form is submit via an h:commandButton or ace:pushButton without ever placing focus on the maskedEntry to render the mask, the label becomes the submitted value. See screenshots 2 and 3.
      Screenshot 2 shows form before submit, inField label displayed properly
      Screenshot 3 shows form that was submit without placing focus on maskedEntry. Note the inField label has been replaced by the value of "Label"

      To reproduce:
       - Use the test app /ICE-11305.jsf found at http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/maskedEntry
       - Use the "MaskedEntry no ajax" form at the top of the page.
       - Change the labelPosition to inField.
       - DO NOT place focus in the maskedEntry
       - Press the Submit button, the label text becomes the value.


      1. 1.PNG
        8 kB
      2. 2.PNG
        8 kB
      3. 3.PNG
        9 kB

        Activity

        Hide
        Mircea Toma added a comment -

        1) Added keypress event handler that verifies input's value before a submit triggered by 'Enter' key.

        2) I cannot reproduce the described issue.

        Show
        Mircea Toma added a comment - 1) Added keypress event handler that verifies input's value before a submit triggered by 'Enter' key. 2) I cannot reproduce the described issue.
        Hide
        Liana Munroe added a comment -

        Verified issue 1 resolved ICEfaces 4 trunk r51549, tomcat 8, MS edge, IE 11, 10 ,9, FF 47, Chrome 57.

        Issue 2 can be reproduced in any browser. Please follow these step by step instructions:
        To reproduce:

        Show
        Liana Munroe added a comment - Verified issue 1 resolved ICEfaces 4 trunk r51549, tomcat 8, MS edge, IE 11, 10 ,9, FF 47, Chrome 57. Issue 2 can be reproduced in any browser. Please follow these step by step instructions: To reproduce: Use the test app / ICE-11305 .jsf found at http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/maskedEntry Use the "MaskedEntry no ajax" form at the top of the page. Change the labelPosition to inField. DO NOT place focus in the maskedEntry Press the Submit button, the label text becomes the value.
        Hide
        Mircea Toma added a comment -

        I followed the steps you recommended but still could not reproduce it. Are you sure you have everything up to date?

        Show
        Mircea Toma added a comment - I followed the steps you recommended but still could not reproduce it. Are you sure you have everything up to date?
        Hide
        Mircea Toma added a comment -

        Never mind, I can reproduce it now if I use Safari. I still cannot reproduce the issue with Chrome.

        Show
        Mircea Toma added a comment - Never mind, I can reproduce it now if I use Safari. I still cannot reproduce the issue with Chrome.
        Hide
        Mircea Toma added a comment -

        2) The applied fix removes the inField label temporarily and thus setting the input's value to the empty string while the form is submitted.

        Show
        Mircea Toma added a comment - 2) The applied fix removes the inField label temporarily and thus setting the input's value to the empty string while the form is submitted.
        Hide
        Liana Munroe added a comment -

        Verified issue 2 resolved ICEfaces 4 trunk r51558, tomcat 8, MS edge, IE 11, 10, 9, FF 47, Chrome 57.
        Issue 2 still occurs when using IE 8.

        Show
        Liana Munroe added a comment - Verified issue 2 resolved ICEfaces 4 trunk r51558, tomcat 8, MS edge, IE 11, 10, 9, FF 47, Chrome 57. Issue 2 still occurs when using IE 8.
        Hide
        Ken Fyten added a comment -

        There is no obvious solution for the IE8 issue, so we'll consider this a Known Issue for IE8 at this time.

        Marking Resolved.

        Show
        Ken Fyten added a comment - There is no obvious solution for the IE8 issue, so we'll consider this a Known Issue for IE8 at this time. Marking Resolved.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: