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

              Ascending order - Click to sort in descending order
              Liana Munroe created issue -
              Liana Munroe made changes -
              Field Original Value New Value
              Attachment resize1.PNG [ 22569 ]
              Attachment resize2.PNG [ 22570 ]
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Fix Version/s 4.3 [ 13096 ]
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51910 Mon Sep 18 14:57:41 MDT 2017 mircea.toma ICE-11346 Render panel's header, content and footer as rows in a table to ensure that they are always resized together.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/panel/PanelRenderer.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/panel/panel.css
              Hide
              Permalink
              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
              Liana Munroe made changes -
              Attachment align.PNG [ 22607 ]
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51943 Mon Sep 25 17:57:20 MDT 2017 mircea.toma ICE-11346 Adjust panel's title style to use the entire width of the panel.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/panel/panel.css
              Hide
              Permalink
              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
              Liana Munroe made changes -
              Attachment IE10.PNG [ 22609 ]
              Hide
              Permalink
              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
              Permalink
              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
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              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
              Permalink
              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.
              Mircea Toma made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51978 Mon Oct 02 15:35:14 MDT 2017 mircea.toma ICE-11346 Use a DIV element (instead of TD) as the container that's animated.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/panel/PanelRenderer.java
              Hide
              Permalink
              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).
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: