ICEfaces
  1. ICEfaces
  2. ICE-4485

dropTarget=true causes treeNode text to scroll when scrolled

          Details

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

            Description

            When setting dropTarget = true on a panelGroup that surrounds a treeNode and having a scrollable div causes the node contents to scroll outside of the div. The tree is inside of a panelDivider which seems to also play a part in the issue. Screenshots and test case attached.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Tyler Johnson added a comment -

              Test Case intended for deployment on Tomcat 6.

              Show
              Tyler Johnson added a comment - Test Case intended for deployment on Tomcat 6.
              Hide
              Permalink
              yip.ng added a comment -

              Removed the divider panel and the problem still exists. So this problem has nothing to do with the divider panel. Also, if dropTarget is changed to false, the problem disappears. So this problem is solely caused by the tree nodes being a drop target inside a scrollable div.

              Show
              yip.ng added a comment - Removed the divider panel and the problem still exists. So this problem has nothing to do with the divider panel. Also, if dropTarget is changed to false, the problem disappears. So this problem is solely caused by the tree nodes being a drop target inside a scrollable div.
              Hide
              Permalink
              yip.ng added a comment - - edited

              This is an IE bug. Scriptaculous changes the position property of drop targets. This triggers the IE bug. Ironically, the line in Scriptaculous that triggers this bug has this comment: "fix IE".

              The fix for this bug is either use quirks mode or set the position of the scrolling div to relative. See:

              http://mt-olympus.com/emmett/bug_overflow_positionrelative.php
              http://rowanw.com/bugs/overflow_relative.htm

              These fixes work with the simple stripped down test case, but they don't seem to work with the original test case with more complicated layout.

              Show
              yip.ng added a comment - - edited This is an IE bug. Scriptaculous changes the position property of drop targets. This triggers the IE bug. Ironically, the line in Scriptaculous that triggers this bug has this comment: "fix IE". The fix for this bug is either use quirks mode or set the position of the scrolling div to relative. See: http://mt-olympus.com/emmett/bug_overflow_positionrelative.php http://rowanw.com/bugs/overflow_relative.htm These fixes work with the simple stripped down test case, but they don't seem to work with the original test case with more complicated layout.
              Hide
              Permalink
              yip.ng added a comment -

              To work around this, add the following style properties to treeContainer:

              height:330px; overflow:auto; position:relative

              See the attached ScreenHunter_01.jpg for the diff.; see ScreenHunter_02.jpg for the result.

              The important thing is to make the tree container height not greater than the divider panel height, so that the tree container scroll bar appears instead of the divider panel scroll bar. Remember to include the padding in the height calculation.

              Show
              yip.ng added a comment - To work around this, add the following style properties to treeContainer: height:330px; overflow:auto; position:relative See the attached ScreenHunter_01.jpg for the diff.; see ScreenHunter_02.jpg for the result. The important thing is to make the tree container height not greater than the divider panel height, so that the tree container scroll bar appears instead of the divider panel scroll bar. Remember to include the padding in the height calculation.

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Tyler Johnson
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: