ICEfaces
  1. ICEfaces
  2. ICE-10841

ace:dateTimeEntry - dynamic change of styleClass prevents popup from opening

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-4.0.0.GA, EE-3.3.0.GA_P03
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      An ace:dateTimeEntry is set to change some styling based on if the selected date is the current day or not. When this change is made, the styling is applied correctly but the popup isn't displayed after this change.

      In Chrome I saw the following error on some tests:
      Uncaught TypeError: Cannot read property 'dontShowPopup' of undefined

        Activity

        Arran Mccullough created issue -
        Hide
        Arran Mccullough added a comment -

        Attached test case that shows the issue.

        Steps:

        • Load welcomeICEfaces.jsf
        • In the date field, select todays date from the popup.
        • Once selected the border on the input field has changed.
        • Clicking the field no longer will show the popup.
        Show
        Arran Mccullough added a comment - Attached test case that shows the issue. Steps: Load welcomeICEfaces.jsf In the date field, select todays date from the popup. Once selected the border on the input field has changed. Clicking the field no longer will show the popup.
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment Case13608Example.war [ 21985 ]
        Attachment Case13608Example.zip [ 21986 ]
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s 4.1 [ 11375 ]
        Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
        Assignee Priority P2 [ 10011 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #46232 Tue Nov 17 11:03:23 MST 2015 art.zambrano ICE-10841 added mechanism to keep track of previously initialized ace:dateTimeEntry instances in the client and initialize them right away when the component is recreated in the client, instead of doing lazy initialization, in order to re-apply all the event listeners necessary to keep the component behaving as expected after a full-markup update
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/datetimeentry.js
        Hide
        Arturo Zambrano added a comment -

        r46232: added mechanism to keep track of previously initialized ace:dateTimeEntry instances in the client and initialize them right away when the component is recreated in the client, instead of doing lazy initialization, in order to re-apply all the event listeners necessary to keep the component behaving as expected after a full-markup update

        Show
        Arturo Zambrano added a comment - r46232: added mechanism to keep track of previously initialized ace:dateTimeEntry instances in the client and initialize them right away when the component is recreated in the client, instead of doing lazy initialization, in order to re-apply all the event listeners necessary to keep the component behaving as expected after a full-markup update
        Hide
        Arturo Zambrano added a comment -

        r46234: committed fix to 3.3 EE maintenance branch

        Show
        Arturo Zambrano added a comment - r46234: committed fix to 3.3 EE maintenance branch
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r46236. Tomcat 7, IE 11, 10, 9, ,8, 7, Ff 34, Chrome 45.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r46236. Tomcat 7, IE 11, 10, 9, ,8, 7, Ff 34, Chrome 45.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: