ICEfaces
  1. ICEfaces
  2. ICE-6869

ACE Calendar doesn't display in pop-up mode when running as a portlet and ace:animation portlet is present on the page

    Details

      Description

      When running the ACE Calendar component as a portlet, the popup does not display when you click the calendar icon. Switching to non pop-up mode displays the calendar and it's fully functional. Switching back to pop-up mode and it still does not display.

        Activity

        Hide
        Deryk Sinotte added a comment -

        The DateTime failure only seems to occur when DateTime is on the same portal page as the Animation portlet. When that happens, Chrome console shows the following JS error:

        ace1:1Uncaught TypeError: Cannot call method 'cloneNode' of null
        eace1:1
        gace1:1
        (anonymous function)ace1:1
        ice.yui3.useace1:1
        ice.animation.registerace1:1
        ice.animation.getAnimationace1:1
        (anonymous function)ace1:1
        E.Subscriber._notifyace1:8
        E.Subscriber.notifyace1:9
        E.CustomEvent._notifyace1:8
        I.bindace1:8

        Show
        Deryk Sinotte added a comment - The DateTime failure only seems to occur when DateTime is on the same portal page as the Animation portlet. When that happens, Chrome console shows the following JS error: ace1:1Uncaught TypeError: Cannot call method 'cloneNode' of null eace1:1 gace1:1 (anonymous function)ace1:1 ice.yui3.useace1:1 ice.animation.registerace1:1 ice.animation.getAnimationace1:1 (anonymous function)ace1:1 E.Subscriber._notifyace1:8 E.Subscriber.notifyace1:9 E.CustomEvent._notifyace1:8 I.bindace1:8
        Hide
        Arturo Zambrano added a comment -

        The calendar code tries to animate the opening/closing of the popup. There was a problem when trying to access the nodes to be animated, since the Y.one() function is used in the animation code. This function also accepts selectors, so when one passes an ID string that contains colons, it could be interpreted as a selector and not as an ID. This is solved by escaping all colons in the string. The problem in this case was that only one occurrence of a colon in the string was being escaped, so it wasn't possible to find nodes that had two or more colons in their IDs.

        Committed fix to 2.0 maintenance branch to revision 24571.
        (this fix along with other related fixes and refactoring will be committed to the 2.0 trunk as part of ICE-6860)

        Show
        Arturo Zambrano added a comment - The calendar code tries to animate the opening/closing of the popup. There was a problem when trying to access the nodes to be animated, since the Y.one() function is used in the animation code. This function also accepts selectors, so when one passes an ID string that contains colons, it could be interpreted as a selector and not as an ID. This is solved by escaping all colons in the string. The problem in this case was that only one occurrence of a colon in the string was being escaped, so it wasn't possible to find nodes that had two or more colons in their IDs. Committed fix to 2.0 maintenance branch to revision 24571. (this fix along with other related fixes and refactoring will be committed to the 2.0 trunk as part of ICE-6860 )

          People

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

            Dates

            • Created:
              Updated:
              Resolved: