ICEfaces
  1. ICEfaces
  2. ICE-9345

CLONE - ace:chart - 'Invalid Argument' JS in IE7

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      EE 3.3 Build 10, IE7, ICEfaces Showcase Chart Overview,
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration
    • Workaround Description:
      Give fixed dimensions to every ace:chart that might be rendered on IE7 / IE8

      Description

      "Invalid Argument" JS errors appear when running the showcase ace:chart demos on IE7 for the following demos (other chart demos are working correctly in IE7):

      - Overview
      - Bar
      - Bubble
      - Gauge
      - Line
      - Combined
      - Dynamic

        Activity

        Hide
        Ken Fyten added a comment -

        Seems similar to the older ICE-8703 issue but also happens on the Overview page, which is navigated to directly?

        Show
        Ken Fyten added a comment - Seems similar to the older ICE-8703 issue but also happens on the Overview page, which is navigated to directly?
        Hide
        Nils Lundquist added a comment -

        If this is not occurring on IE8 also it is not related to the compatibility library required for ace:chart. This would mean that this is some new IE7 specific failure.

        Show
        Nils Lundquist added a comment - If this is not occurring on IE8 also it is not related to the compatibility library required for ace:chart. This would mean that this is some new IE7 specific failure.
        Hide
        Nils Lundquist added a comment -

        This issue is distinct from ICE-8703 because it cannot be reproduced in IE8.

        Show
        Nils Lundquist added a comment - This issue is distinct from ICE-8703 because it cannot be reproduced in IE8.
        Hide
        Nils Lundquist added a comment - - edited

        Rarely does this issue causes an 'invalid argument', more often it causes the following error:

        SCRIPT5007: Unable to get value of the property 'initElement': object is null or undefined
        ace-chart.uncompressed.js.jsf?ln=icefaces.ace&v=3_4_0_130617, line 297 character 17
        Could not complete the operation due to error 80020101.

        Attempting to catch execution with a breakpoint on that line always causes IE to crash.

        Show
        Nils Lundquist added a comment - - edited Rarely does this issue causes an 'invalid argument', more often it causes the following error: SCRIPT5007: Unable to get value of the property 'initElement': object is null or undefined ace-chart.uncompressed.js.jsf?ln=icefaces.ace&v=3_4_0_130617, line 297 character 17 Could not complete the operation due to error 80020101. Attempting to catch execution with a breakpoint on that line always causes IE to crash.
        Hide
        Nils Lundquist added a comment - - edited

        The code at that line is:
        this.initCanvas = function(canvas) {
        if ($.jqplot.use_excanvas)

        { return window.G_vmlCanvasManager.initElement(canvas); }

        return canvas;
        };

        The excanvas compat library is failing initialization / parsing on IE7 only.

        Show
        Nils Lundquist added a comment - - edited The code at that line is: this.initCanvas = function(canvas) { if ($.jqplot.use_excanvas) { return window.G_vmlCanvasManager.initElement(canvas); } return canvas; }; The excanvas compat library is failing initialization / parsing on IE7 only.
        Hide
        Nils Lundquist added a comment -

        This is resulting from a somewhat-known timing issue in IE7:
        http://stackoverflow.com/questions/6589307/ie7-excanvas-document-ready-and-ie7-creates-problem

        The only suggestion to workaround it I've found is that all chart initializations must occur at body onLoad - of course, that is impossible for an ICEfaces application as onLoad is only called when a GET request is completed.

        There may very well not be any solution to this.

        Show
        Nils Lundquist added a comment - This is resulting from a somewhat-known timing issue in IE7: http://stackoverflow.com/questions/6589307/ie7-excanvas-document-ready-and-ie7-creates-problem The only suggestion to workaround it I've found is that all chart initializations must occur at body onLoad - of course, that is impossible for an ICEfaces application as onLoad is only called when a GET request is completed. There may very well not be any solution to this.
        Hide
        Ken Fyten added a comment -

        Due to the complex nature of the issue (doesn't occur all the time, depends on unknown combination of factors in the application), and the fact that there is no reliable solution that could readily be applied to the ICEfaces use-cases, marking this as Won't Fix and considering this a Known Issue for the ace:chart component when running in IE7.

        Show
        Ken Fyten added a comment - Due to the complex nature of the issue (doesn't occur all the time, depends on unknown combination of factors in the application), and the fact that there is no reliable solution that could readily be applied to the ICEfaces use-cases, marking this as Won't Fix and considering this a Known Issue for the ace:chart component when running in IE7.
        Hide
        Nils Lundquist added a comment -

        Removing the style classes from the ace:panel content div resolves this issue. Why this is the case is undetermined.

        Show
        Nils Lundquist added a comment - Removing the style classes from the ace:panel content div resolves this issue. Why this is the case is undetermined.
        Hide
        Nils Lundquist added a comment -

        Revision #36285
        Committed by nils.lundquist
        Moments ago
        ICE-9345 - Removed ace:panel from content-template.xhtml to fix ace:chart on IE7

        Show
        Nils Lundquist added a comment - Revision #36285 Committed by nils.lundquist Moments ago ICE-9345 - Removed ace:panel from content-template.xhtml to fix ace:chart on IE7
        Hide
        Liana Munroe added a comment -

        chart - Dynamic Chart IE-7
        When clicking on a cell or row in the table the Letter Axis for the chart bottom does not render entirely. (screen)

        Show
        Liana Munroe added a comment - chart - Dynamic Chart IE-7 When clicking on a cell or row in the table the Letter Axis for the chart bottom does not render entirely. (screen)
        Hide
        Ken Fyten added a comment -

        Last issue has been spun off into a separate JIRA: ICE-9375

        Show
        Ken Fyten added a comment - Last issue has been spun off into a separate JIRA: ICE-9375

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Nils Lundquist
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: