ICEfaces
  1. ICEfaces
  2. ICE-9456

ICEfaces 1.8 component-showcase [jsp] - Panel: Split Pane Panel demo JS errors

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P03
          • Fix Version/s: EE-1.8.2.GA_P07
          • Component/s: ICE-Components, Sample Apps
          • Labels:
            None
          • Environment:
            ICEfaces EE 1.8.2.GA_P07 Build5
            App Server: Tomcat6
            Browsers: IE9/10, Chrome27, FF21 (not an issue in older IE browsers)
            Sample app: component-showcase [jsp]
          • Assignee Priority:
            P1

            Description

            On Panel: Split Pane Panel demo - a JS error occurs on page load and when dragging the divider.

            IE10 error:
            [window.voUF#44] failed to insert element: <script id="voUFl7D0TCDPW464KWCIrw:44:dynamic-code" type="text/javascript">...</script>
            TypeError: Unable to get property 'style' of undefined or null reference

            Chrome27 error:
            [window.TomB#1] failed to insert element: <script id="TomBMHNIyJrYhlOQV4F7wg:1:dynamic-code" type="text/javascript">...</script> TypeError {} icefaces-d2d.js:14

              Activity

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Carmen Cristurean made changes -
              Field Original Value New Value
              Summary ICEfaces 1.8 componet-showcase [jsp] - Split Pane Panel demo JS errors ICEfaces 1.8 component-showcase [jsp] - Panel: Split Pane Panel demo JS errors
              Carmen Cristurean made changes -
              Environment ICEfaces EE 1.8.2.GA_P07 Build5
              App Server: Tomcat6
              Browsers: IE9/10, Chrome27, FF21 (not an issue in older IE browsers)               		ICEfaces EE 1.8.2.GA_P07 Build5
              App Server: Tomcat6
              Browsers: IE9/10, Chrome27, FF21 (not an issue in older IE browsers)
              Sample app: component-showcase [jsp]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-1.8.2.GA_P07 [ 10672 ]
              Affects Version/s EE-1.8.2.GA_P07 [ 10672 ]
              Assignee Priority P1 [ 10010 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #37213 Tue Jul 23 16:00:45 MDT 2013 art.zambrano ICE-9456 fixed the way to infer the panel divider container node's client id, so it doesn't produce NPE's when part of the client id contains the 'Divider' string somewhere else
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/resizablePanelGrid.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix to trunk at revision 37213 and to the p07 tag at revision 37214.

              The solution consists in fixing the way the panel divider container node's id is inferred, so that the client-side code accurately selects the node and acts on it, without producing NPE's. The issue was the fact that the container id was determined by removing the 'Divider' string at the end of the client id that was passed to the adjustPercentBasedHeight() function. However, this code wasn't using a regular expression to ensure that this string should be removed only from the end of the client id, so problems occurred when the client id contained the word 'Divider' at some other point.

              Show
              Arturo Zambrano added a comment - Committed fix to trunk at revision 37213 and to the p07 tag at revision 37214. The solution consists in fixing the way the panel divider container node's id is inferred, so that the client-side code accurately selects the node and acts on it, without producing NPE's. The issue was the fact that the container id was determined by removing the 'Divider' string at the end of the client id that was passed to the adjustPercentBasedHeight() function. However, this code wasn't using a regular expression to ensure that this string should be removed only from the end of the client id, so problems occurred when the client id contained the word 'Divider' at some other point.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Confirmed as resolved with ICEfaces 1.8.2.GA_P07 Build8, in IE6/7/8/9/10/FF21/Chrome27.

              Show
              Carmen Cristurean added a comment - Confirmed as resolved with ICEfaces 1.8.2.GA_P07 Build8, in IE6/7/8/9/10/FF21/Chrome27.
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Affects Version/s EE-1.8.2.GA_P03 [ 10251 ]
              Affects Version/s EE-1.8.2.GA_P07 [ 10672 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: