Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 2.1, Advanced Component Environment (ace:)
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration

      Description

      Create a new ace:accordion (panel) component with display, behavior and performance customization options.

      1. accordionAceTabSet-IE8.jpg
        188 kB
      2. accordionCompatPanelTabSet-IE8.jpg
        163 kB
      3. accordionInDataTbl-IE8.jpg
        168 kB
      4. accordionRepeatDtTable1-IE8.jpg
        195 kB
      5. accordionRepeatDtTable2-IE8.jpg
        196 kB
      6. accordionRepeat-IE8.jpg
        186 kB
      7. collapsible-IE8.jpg
        168 kB
      8. Push1.jpg
        65 kB
      9. Push2.jpg
        141 kB

        Activity

        Hide
        yip.ng added a comment -

        The JS error still present in IE8 (even in standards mode) is caused by animation. Somehow after a bunch of animation calculations a negative height is being set in IE8, which triggers the error. There is a comment in the code which says, "IE gets very inconsistent results when animating elements". In Firebug we can see a strange number involved in the calculation: -0.4333999999999989. So this is likely caused by some freak calculation and/or rounding errors in this particular layout scenario.

        Show
        yip.ng added a comment - The JS error still present in IE8 (even in standards mode) is caused by animation. Somehow after a bunch of animation calculations a negative height is being set in IE8, which triggers the error. There is a comment in the code which says, "IE gets very inconsistent results when animating elements". In Firebug we can see a strange number involved in the calculation: -0.4333999999999989. So this is likely caused by some freak calculation and/or rounding errors in this particular layout scenario.
        Hide
        Ken Fyten added a comment -

        Can we modify the JS for the animation to trap the negative height and set it to zero instead then?

        Show
        Ken Fyten added a comment - Can we modify the JS for the animation to trap the negative height and set it to zero instead then?
        Hide
        yip.ng added a comment -

        Done and tested.

        Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js
        Completed: At revision: 26652

        Show
        yip.ng added a comment - Done and tested. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js Completed: At revision: 26652
        Hide
        Carmen Cristurean added a comment -

        Tested ace:accordion component using code revision 26767 with IE8 .

        There are no more JS errors, however the accordion inside ace:tabSet does not work properly: if switching to the closed panes, the accordion collapses altogether and panes cannot be open afterwards.
        This issue occurs in IE 8 and Chrome 15 (not in Firefox).

        Unsure if this issue is related to the fix for the JS errors in IE8 (this JIRA), or to the fix for the 'collapsible' attribute (ICE-7497).

        Note: The accordion component on the test page was defined with collapsible="false".

        To reproduce:

        Show
        Carmen Cristurean added a comment - Tested ace:accordion component using code revision 26767 with IE8 . There are no more JS errors, however the accordion inside ace:tabSet does not work properly: if switching to the closed panes, the accordion collapses altogether and panes cannot be open afterwards. This issue occurs in IE 8 and Chrome 15 (not in Firefox). Unsure if this issue is related to the fix for the JS errors in IE8 (this JIRA), or to the fix for the 'collapsible' attribute (ICE-7497). Note: The accordion component on the test page was defined with collapsible="false". To reproduce: test application location: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/accordion load the test page for accordion isnide ace:tabSet: http://localhost:8080/accordion/accordionIceTabSet.jsf page opens with first pane open. Click on the next pane to open -> this works fine. click on the third pane -> this collapses all accordionPanes.
        Hide
        Carmen Cristurean added a comment -

        Created ICE-7546 for the ace:accordion inside ace:tabSet issue described above, and closing this as per Ken's request.

        Show
        Carmen Cristurean added a comment - Created ICE-7546 for the ace:accordion inside ace:tabSet issue described above, and closing this as per Ken's request.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: