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

        Hide
        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
        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
        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
        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
        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: