ICEfaces
  1. ICEfaces
  2. ICE-11346

ace:resizable panel in iterations issue

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2, EE-4.2.0.GA
    • Fix Version/s: 4.3
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Tomcat 7/8, ICEfaces 4.2 release and up, all browsers.

      Description

      The ace:resizable QA test apps where the ace:resizable is inside a table or repeat have a containment issue. This was not seen in IF 4.1 but was found in 4.2.
      See attached screen shots.

      resize1.PNG - ICEfaces 4.1 - In the third row I resized the panel vertically. After the resize the words "ICEsoft Technologies" stayed within the panel.

      resize2.PNG - ICEfaces 4.2 and current - In the third row I resized the panel vertically. After the resize the words "ICEsoft Technologies" dropped out of the panel.

      1. align.PNG
        21 kB
      2. IE10.PNG
        52 kB
      3. resize1.PNG
        23 kB
      4. resize2.PNG
        25 kB

        Activity

        Hide
        Mircea Toma added a comment - - edited

        Render panel's header, content and footer as rows in a table to ensure that they are always resized together with a bounding box around all the panel parts.

        Show
        Mircea Toma added a comment - - edited Render panel's header, content and footer as rows in a table to ensure that they are always resized together with a bounding box around all the panel parts.
        Hide
        Liana Munroe added a comment - - edited

        Verified original issue ICEfaces 4 trunk r51921, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 53, Chrome 60.
        This fix has caused a regression in the showcase demo layout. The documentation menu is no longer right aligned in the demo header. See attached screen shot: align.png

        Show
        Liana Munroe added a comment - - edited Verified original issue ICEfaces 4 trunk r51921, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 53, Chrome 60. This fix has caused a regression in the showcase demo layout. The documentation menu is no longer right aligned in the demo header. See attached screen shot: align.png
        Hide
        Mircea Toma added a comment -

        Adjust panel's title style to use the entire width of the panel.

        Show
        Mircea Toma added a comment - Adjust panel's title style to use the entire width of the panel.
        Hide
        Liana Munroe added a comment - - edited

        Tested with ICEfaces 4 trunk r51943, Tomcat 8. The showcase style adjustment was successful when viewed in Chrome 61, FF 53 and MS Edge as well as IE 8 through 11 when viewed on a windows 10 machine using Emulation in an MS Edge browser.

        A style issue persists when viewing the showcase demo on IE 8 through 11 in a non emulated mode on the QA test machines, or in emulation from MS Edge on a windows 8 machine.
        Please see attached screen shot IE10.PNG

        Show
        Liana Munroe added a comment - - edited Tested with ICEfaces 4 trunk r51943, Tomcat 8. The showcase style adjustment was successful when viewed in Chrome 61, FF 53 and MS Edge as well as IE 8 through 11 when viewed on a windows 10 machine using Emulation in an MS Edge browser. A style issue persists when viewing the showcase demo on IE 8 through 11 in a non emulated mode on the QA test machines, or in emulation from MS Edge on a windows 8 machine. Please see attached screen shot IE10.PNG
        Hide
        Ken Fyten added a comment -

        Let's consider this minor IE8-10 styling issue, that is specific to the showcase sample app a Known Issue.

        Show
        Ken Fyten added a comment - Let's consider this minor IE8-10 styling issue, that is specific to the showcase sample app a Known Issue.
        Hide
        Liana Munroe added a comment -

        It was discovered while verifying ICE-11330 that the commit made for this JIRA at r51910 causes the panel expansion and contraction animation to stutter. This can be demonstrated by comparing the Jenkins ICEfaces 4 trunk continuous build #2663 showcase ace:panel expansion and contraction motions against the same demo in the public showcase at http://icefaces-showcase.icesoft.org/showcase.jsf?grp=ace:panel

        Show
        Liana Munroe added a comment - It was discovered while verifying ICE-11330 that the commit made for this JIRA at r51910 causes the panel expansion and contraction animation to stutter. This can be demonstrated by comparing the Jenkins ICEfaces 4 trunk continuous build #2663 showcase ace:panel expansion and contraction motions against the same demo in the public showcase at http://icefaces-showcase.icesoft.org/showcase.jsf?grp=ace:panel
        Hide
        Ken Fyten added a comment -

        It would appear that the panel contents are no longer able to be resized by the animation, so the animated close goes from 95% to 0% open in one step. No good.

        Show
        Ken Fyten added a comment - It would appear that the panel contents are no longer able to be resized by the animation, so the animated close goes from 95% to 0% open in one step. No good.
        Hide
        Mircea Toma added a comment -

        Use a DIV element (instead of TD) as the container that's being animated.

        Show
        Mircea Toma added a comment - Use a DIV element (instead of TD) as the container that's being animated.
        Hide
        Liana Munroe added a comment -

        Verified panel animation resolved ICEfaces 4 trunk r51981, Tomcat 8, MS Edge, IE 11, 10, 9, 8 Chrome 61, FF 53.
        Note styling issue still exists in showcase app (as shown in screen shot IE10.PNG) when using IE 8 through 11 (non-emulated).

        Show
        Liana Munroe added a comment - Verified panel animation resolved ICEfaces 4 trunk r51981, Tomcat 8, MS Edge, IE 11, 10, 9, 8 Chrome 61, FF 53. Note styling issue still exists in showcase app (as shown in screen shot IE10.PNG) when using IE 8 through 11 (non-emulated).

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Liana Munroe
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: