ICEfaces
  1. ICEfaces
  2. ICE-10583

ace:resizable does not respect containment attribute

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.0
          • Fix Version/s: 4.2.BETA, 4.2
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 4 r44277/ ICEfaces EE-4.0.0 r41234. Tomcat 7, all browsers
          • Assignee Priority:
            P2
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.)

            Description

            containment=true attribute is not respected. Panel can be dragged outside the parent div boundries.
            Test application /panel/resizableDynAttribute.jsf
            http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/resizable

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Carmen Cristurean added a comment - - edited

              Issue is still reproducible on ICEfaces4 trunk r.48632, or EE4 trunk r48694.

              Show
              Carmen Cristurean added a comment - - edited Issue is still reproducible on ICEfaces4 trunk r.48632, or EE4 trunk r48694.
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              The resizable feature seems to be running properly. The problem is caused by how the ace:panel is rendered. The ace:panel markup has a DIV as root element and another enclosed DIV as the content are for the panel. When the resizable feature is use the root DIV for the containment calculation, while its resized content is enclosed by the content DIV. Since both the root DIV and content _DIV have some padding that will shift the resizable ace:panel's content.

              Modifying the following CSS classes will prove that the resize feature works well:

              ui-panel { padding: 0; }
ui-panel-content { padding: 0; }

              or replace the enclosing ace:panel with a DIV:

              <div style="width:300px;height:150px; border: 1px solid black;"> ... </div>
              Show
              Mircea Toma added a comment - - edited The resizable feature seems to be running properly. The problem is caused by how the ace:panel is rendered. The ace:panel markup has a DIV as root element and another enclosed DIV as the content are for the panel. When the resizable feature is use the root DIV for the containment calculation, while its resized content is enclosed by the content DIV . Since both the root DIV and content _DIV have some padding that will shift the resizable ace:panel 's content. Modifying the following CSS classes will prove that the resize feature works well: ui-panel { padding: 0; } ui-panel-content { padding: 0; } or replace the enclosing ace:panel with a DIV : <div style= "width:300px;height:150px; border: 1px solid black;" > ... </div>
              Hide
              Permalink
              Mircea Toma added a comment -

              Marking this as won't fix. See ICE-11197 for explanation.

              Show
              Mircea Toma added a comment - Marking this as won't fix. See ICE-11197 for explanation.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              ICEfaces4 trunk r49785/ MsEdge, FF49, IE11, Chrome55: the original issue is resolved.
              However, if enabling the 'animate' attribute on the resizable component, the panel still extends over the limits when the mouse is moved off the element, after resizing the panel.

              Show
              Carmen Cristurean added a comment - ICEfaces4 trunk r49785/ MsEdge, FF49, IE11, Chrome55: the original issue is resolved. However, if enabling the 'animate' attribute on the resizable component, the panel still extends over the limits when the mouse is moved off the element, after resizing the panel.
              Hide
              Permalink
              Mircea Toma added a comment -

              The problem mentioned above seems to be known, but still not solved by jQuery: https://bugs.jqueryui.com/ticket/4624 .

              Show
              Mircea Toma added a comment - The problem mentioned above seems to be known, but still not solved by jQuery: https://bugs.jqueryui.com/ticket/4624 .

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: