ICEfaces
  1. ICEfaces
  2. ICE-4915

panelTooltip positioning problem on window resize + scroll

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      panelTooltip

      Description

      If you resize the browser window with this open and then scroll the page, the tooltip doesn't position near the mouse correctly anymore.
      1. video-1.swf
        2.24 MB
        yip.ng
      1. screenshot-1.jpg
        214 kB

        Activity

        Adnan Durrani created issue -
        Adnan Durrani made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Adnan Durrani made changes -
        Comment [ A comment with security level 'icesoft-internal-developers' was removed. ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 1.8.2 [ 10190 ]
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
        Hide
        yip.ng added a comment -

        How do you resize and scroll with the tooltip open? The tooltip will just disappear when you move the mouse cursor away.

        Show
        yip.ng added a comment - How do you resize and scroll with the tooltip open? The tooltip will just disappear when you move the mouse cursor away.
        Hide
        Adnan Durrani added a comment -

        Bring the tooltip demo page, resize the browser and scroll down a bit. Then mouse over on any province name (it happens with dynamic tooltip only). You will see the tooltip will not appear at the right position.

        Show
        Adnan Durrani added a comment - Bring the tooltip demo page, resize the browser and scroll down a bit. Then mouse over on any province name (it happens with dynamic tooltip only). You will see the tooltip will not appear at the right position.
        yip.ng made changes -
        Attachment screenshot-1.jpg [ 11981 ]
        Hide
        yip.ng added a comment -

        There are actually at least two causes for the problems.

        First, there is some erratic and excessive scrolling whenever you are at the bottom of the page. This is caused by the fix for ICE-3720. This problem had been there for as long as the fix for ICE-3720 was in. See screenshot-1 for the code change for ICE-3720. We need to revert this change (at least temporarily) to eliminate its interference.

        Second, tooltip positioning within viewport was actually done already in ICE-3736. The code was later modified to accommodate the case in ICE-3745. This modification doesn't seem to work anymore after the tooltip positioning improvements in ICE-4853.

        So, we need to find a way to make it work for all the above 3 cases.

        Show
        yip.ng added a comment - There are actually at least two causes for the problems. First, there is some erratic and excessive scrolling whenever you are at the bottom of the page. This is caused by the fix for ICE-3720 . This problem had been there for as long as the fix for ICE-3720 was in. See screenshot-1 for the code change for ICE-3720 . We need to revert this change (at least temporarily) to eliminate its interference. Second, tooltip positioning within viewport was actually done already in ICE-3736 . The code was later modified to accommodate the case in ICE-3745 . This modification doesn't seem to work anymore after the tooltip positioning improvements in ICE-4853 . So, we need to find a way to make it work for all the above 3 cases.
        Hide
        yip.ng added a comment -
        Show
        yip.ng added a comment - http://screencast.com/t/uMe1fLlZ8E
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #19230 Tue Sep 08 12:39:45 MDT 2009 yip.ng ICE-4915: Fixed wrong positioning of tooltips when window is resized to have scrollbars.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/tooltip_panelpopup.js
        yip.ng made changes -
        Attachment video-1.swf [ 11982 ]
        Hide
        yip.ng added a comment -

        Fixed. See above video link and also attached video-1.

        Show
        yip.ng added a comment - Fixed. See above video link and also attached video-1.
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 1.8.2-RC1 [ 10210 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P1

          People

          • Assignee:
            yip.ng
            Reporter:
            Adnan Durrani
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: