ICEfaces
  1. ICEfaces
  2. ICE-3457

panelPositioned in scollable div causes unexpected behavior

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.1
    • Fix Version/s: 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When a panelPosition is located inside a scrollable div there is unexpected behavior with the dragging and dropping of items. There is no problem when the div is scrolled to the top but if you scroll to the bottom and try and drag an item the empty space it renders is not in the correct position. It looks like the drag and drop mechanism doesn't take into account the scrolled offset.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached screen shot of issue, test case code, and built war file for testcase (Built to run on NetBeans 6.x).

        Show
        Arran Mccullough added a comment - Attached screen shot of issue, test case code, and built war file for testcase (Built to run on NetBeans 6.x).
        Hide
        Arran Mccullough added a comment -

        Comment from support customer:

        The strange behavior also occurs when the panelPositioned is placed in a DIV (container) that is not scrollable but this DIV is placed in a parent DIV that is scrollable. So you could have:

        DIV (scrollable content)

        __ice:PanelGroup (fixed content)
        __ice:panelCollapsible (fixed content)
        __ice:panelPositioned

        I guess your developers should care about the offsets of all scrollable parent containers in the hierarchy not only the immediate parent DIV.

        Show
        Arran Mccullough added a comment - Comment from support customer: The strange behavior also occurs when the panelPositioned is placed in a DIV (container) that is not scrollable but this DIV is placed in a parent DIV that is scrollable. So you could have: DIV (scrollable content) __ice:PanelGroup (fixed content) __ice:panelCollapsible (fixed content) __ice:panelPositioned I guess your developers should care about the offsets of all scrollable parent containers in the hierarchy not only the immediate parent DIV.
        Hide
        Ken Fyten added a comment -

        Similar issue to ICE-3287.

        Show
        Ken Fyten added a comment - Similar issue to ICE-3287 .
        Hide
        John Genoese added a comment -

        This issue is an absolute show-stopper for me. I need to drop various GUI objects on an <ice:tree/>, which is embedded in a <div/> with "overflow:scroll". The drop target tree node is always identified as if the tree view <div/> is scrolled to the top. I will happily attach the test case I prepared, if requested.

        I apologize for pushiness, but can anyone provide a time-frame for the fix? I need to know because a critical portion of my app is completely dependent on this working correctly.

        Show
        John Genoese added a comment - This issue is an absolute show-stopper for me. I need to drop various GUI objects on an <ice:tree/>, which is embedded in a <div/> with "overflow:scroll". The drop target tree node is always identified as if the tree view <div/> is scrolled to the top. I will happily attach the test case I prepared, if requested. I apologize for pushiness, but can anyone provide a time-frame for the fix? I need to know because a critical portion of my app is completely dependent on this working correctly.
        Hide
        Arturo Zambrano added a comment - - edited

        fixed at revision 18631

        EDIT: It's important to note that this fix lies in the original scriptaculous code. We need to make sure to include this fix whenever we update the scriptaculous library (unless this issue comes already solved in new versions).

        Show
        Arturo Zambrano added a comment - - edited fixed at revision 18631 EDIT: It's important to note that this fix lies in the original scriptaculous code. We need to make sure to include this fix whenever we update the scriptaculous library (unless this issue comes already solved in new versions).
        Hide
        Shobhit Gupta added a comment -

        Is this issue really solved in 1.8 version ?

        Show
        Shobhit Gupta added a comment - Is this issue really solved in 1.8 version ?
        Hide
        Joanne Bai added a comment -

        QA verified the fix again on ICEfaces trunk revision 18864

        Test app has been added to the QA repo

        Show
        Joanne Bai added a comment - QA verified the fix again on ICEfaces trunk revision 18864 Test app has been added to the QA repo

          People

          • Assignee:
            Unassigned
            Reporter:
            Arran Mccullough
          • Votes:
            2 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: