ICEfaces
  1. ICEfaces
  2. ICE-7633

ace:resizable: ace:ajax 'resize' event not being triggered

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.0.RC2
          • Fix Version/s: 3.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
          • Assignee Priority:
            P2

            Description

            The ace:ajax event 'resize' is not being triggered when resizing a component (a panel is used in this case). When this event is used in combination with the resizeListener, neither of them are triggered. However, when the resizeListener is by itself it functions fine.

            To reproduce:
            1. Build / deploy test application.
            2. Navigate to 'Resizable Ajax ' test page.
            3. Resize the panel inside the '"resize" Event Test'
            Notice that the actual result is not what is expected. There are some print statements in the ajaxListener & resizeListener for debugging as well.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment -

              Three causes:

              1. In the original "Grimlock" code, a flag is set to trigger event only when there is listener specified by attribute. (<ajax> event was added by us.)
              2. Side effect caused by ICE-7584, resulting in exception shown in screenshot-1.png.
              3. Same problem as ICE-7588. (But again fix for ICE-7588 had side effect. Must use re-fix by Deryk in ICE-7601.)

              Show
              yip.ng added a comment - Three causes: 1. In the original "Grimlock" code, a flag is set to trigger event only when there is listener specified by attribute. (<ajax> event was added by us.) 2. Side effect caused by ICE-7584 , resulting in exception shown in screenshot-1.png. 3. Same problem as ICE-7588 . (But again fix for ICE-7588 had side effect. Must use re-fix by Deryk in ICE-7601 .)
              Hide
              Permalink
              yip.ng added a comment - - edited

              Fixed. See screenshot-2.png.

              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\resizable\resizable.js
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\Resizable.java
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\ResizableMeta.java
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\ResizableRenderer.java
              Completed: At revision: 27114

              Show
              yip.ng added a comment - - edited Fixed. See screenshot-2.png. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\resizable\resizable.js Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\Resizable.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\ResizableMeta.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\resizable\ResizableRenderer.java Completed: At revision: 27114
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Fix was verified with code revision 27114 in IE8, FF3.6, Chrome15.

              Show
              Carmen Cristurean added a comment - Fix was verified with code revision 27114 in IE8, FF3.6, Chrome15.

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Cruz Miraback
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: