ICEfaces
  1. ICEfaces
  2. ICE-8703

ace:chart - 'Invalid Argument' JS err when loaded via partial request in IE7 & 8

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: EE-3.2.0.GA, 3.3
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      IF 3.2
    • 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


      Our current understanding is that when an ace:chart is rendered onto the page with a partial request, and followed by a sibling element in its parent form, in IE 7 & 8, an Invalid Argument javascript exception is thrown from a nonsense location (at least according to the IE debug tools).

      This is capturing the vague issue responsible for ICE-8670

        Activity

        Migration created issue -
        Migration made changes -
        Field Original Value New Value
        Reporter Migration [ remote ] Nils Lundquist [ nils.lundquist ]
        Migration made changes -
        Assignee Nils Lundquist [ nils.lundquist ]
        Ken Fyten made changes -
        Fix Version/s EE-3.2.0.GA [ 10332 ]
        Fix Version/s 3.3 [ 10370 ]
        Hide
        Nils Lundquist added a comment - - edited

        Attempts to reproduce this issue in an isolated test case have been unsuccessful, although it still presents if the alterations to the showcase made in ICE-8670 (rev.31693) to work around this issue are removed. I'm at a loss for how to debug this issue at this point.

        Show
        Nils Lundquist added a comment - - edited Attempts to reproduce this issue in an isolated test case have been unsuccessful, although it still presents if the alterations to the showcase made in ICE-8670 (rev.31693) to work around this issue are removed. I'm at a loss for how to debug this issue at this point.
        Hide
        Nils Lundquist added a comment -

        Reassigning to Ken for delegation.

        Show
        Nils Lundquist added a comment - Reassigning to Ken for delegation.
        Nils Lundquist made changes -
        Assignee Nils Lundquist [ nils.lundquist ] Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Assignee Ken Fyten [ ken.fyten ] Mircea Toma [ mircea.toma ]
        Affects Version/s 3.2 [ 10338 ]
        Assignee Priority P2 [ 10011 ]
        Nils Lundquist made changes -
        Link This issue blocks ICE-8865 [ ICE-8865 ]
        Ken Fyten made changes -
        Assignee Priority P2 [ 10011 ] P1 [ 10010 ]
        Hide
        Mircea Toma added a comment -

        The "Invalid argument" error is thrown in excanvas.js at line 170. The error occurs when a negative value is assigned to el.style.height property. Also, the error is captured in jquery.jplot.js line 227 and rendered in the page as text node.

        I tried to find the reason why a negative height value is acquired off the canvas element but without much success.

        Show
        Mircea Toma added a comment - The "Invalid argument" error is thrown in excanvas.js at line 170. The error occurs when a negative value is assigned to el.style.height property. Also, the error is captured in jquery.jplot.js line 227 and rendered in the page as text node. I tried to find the reason why a negative height value is acquired off the canvas element but without much success.
        Ken Fyten made changes -
        Assignee Mircea Toma [ mircea.toma ] Nils Lundquist [ nils.lundquist ]
        Nils Lundquist made changes -
        Workaround Description Give fixed dimensions to every ace:chart that might be rendered on IE7 / IE8
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33021 Thu Jan 10 11:28:56 MST 2013 nils.lundquist ICE-8703 - Adding height, width and style attributes to Chart to ease styling and workaround use.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/chart/chartExport.xhtml
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/chart/ChartMeta.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/chart/ChartRenderer.java
        Hide
        Nils Lundquist added a comment -

        Revision #33021
        Committed by nils.lundquist
        A minute ago
        ICE-8703 - Adding height, width and style attributes to Chart to ease styling and workaround use.

        Due to the bugged underlying browser behavior and the inconsistency in reproducing this error, the workaround is an acceptable caveat to compatibility use.

        Show
        Nils Lundquist added a comment - Revision #33021 Committed by nils.lundquist A minute ago ICE-8703 - Adding height, width and style attributes to Chart to ease styling and workaround use. Due to the bugged underlying browser behavior and the inconsistency in reproducing this error, the workaround is an acceptable caveat to compatibility use.
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Nils Lundquist made changes -
        Link This issue is duplicated by ICE-8832 [ ICE-8832 ]
        Ken Fyten made changes -
        Affects Documentation (User Guide, Ref. Guide, etc.),Sample App./Tutorial,Compatibility/Configuration [ 10003, 10001, 10002 ]
        Hide
        Jerome Ruzol added a comment -

        Confirmed Fixed as of rvn# 33030

        Show
        Jerome Ruzol added a comment - Confirmed Fixed as of rvn# 33030
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: