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
              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.
              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.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: