ICEfaces
  1. ICEfaces
  2. ICE-3194

ice:menuPopup does not position/render properly when used in combination with ice:panelDivider and ice:tree

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.7
    • Fix Version/s: 1.8RC1, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Icefaces 1.7.0, Facelets, both FF 2.0 and IE 6/7

      Description

      Scenario: my screen is a split panel (ice:panelDivider). The left pane contains a tree (in ice:tree) and right pane shows the content in ice:panelSeries. Now I want to assign a context popup menu to each node of the tree in the left pane.

      Bug manifestation. Upon the right-click on the tree node the menu popup is rendered. But if the menu divs happen to overlap the right pane of split panel (ice:panelDivider) then the overlapped part is corrupted. This effect differs in FF and IE (please see attached images).
      FF: the overlapped part of split panel's right pane shines thru the menu div.
      IE: the right pane of the split panel clips the overlapped part of the menu. That part becomes invisible. Also the positions of the menu divs are shifted.

      By the way, attempts to cure these effects by using 'z-index' do not work.

      Bug description. The concise bug description is given by user 'Pred' in this forum post: http://www.icefaces.org/JForum/posts/list/6583.page. The problem is that the Icefaces has to set menu divs with 'position: absolute'. Within the ice:tree I have to put my ice:menuPopups into ice:panelGroups (see attached menuPopupBug.xhtml). It means that absolutely positioned menu divs become children of tree's internal divs. This would cause problems if they overlapped by any relatively positioned elements on the page.
      I do not have any elements with explicit 'position:relative' on my page. But I suspect that introducing the scrollbars on the divs has the same effect. Note that I set 'height: 30em' on the ice:paneDivider - that may force browsers to consider scrollbars every time they redraw the enclosed divs.
      Anyway, the approach described at http://www.icefaces.org/JForum/posts/list/6583.page worked for my case, but it is too hacky and needs to be moved into the main codebase somehow.

      1. menuPopupBug.xhtml
        2 kB
        Sergey Petrov
      2. MenuPopupBug.java
        2 kB
        Sergey Petrov
      1. right pane of split panel is shining thru the popup menu in FF 2.jpg
        112 kB
      2. popup menu is mispositioned in IE6.jpg
        120 kB
      3. popup menu clipped in IE6.jpg
        116 kB
      4. ICE-3194.jpg
        71 kB

        Issue Links

          Activity

          Hide
          Andy Hoffman added a comment -

          Just found the cloned "open" JIRA issue, and I have added my vote to that bug - please ignore request to re-open this issue, as the cloned issue ICE-4400 is precisely the same issue that I have, just with IE 8.

          Show
          Andy Hoffman added a comment - Just found the cloned "open" JIRA issue, and I have added my vote to that bug - please ignore request to re-open this issue, as the cloned issue ICE-4400 is precisely the same issue that I have, just with IE 8.
          Hide
          Andy Hoffman added a comment -

          This is still an issue also with IE 8. Firefox is OK, but IE 8 is showing the mis-positioned pop-up. Please re-open this issue for testing against IE 8.

          The issue is showing for menuPopup component, nested within tree, nested within panelDivider (facet "first").

          Show
          Andy Hoffman added a comment - This is still an issue also with IE 8. Firefox is OK, but IE 8 is showing the mis-positioned pop-up. Please re-open this issue for testing against IE 8. The issue is showing for menuPopup component, nested within tree, nested within panelDivider (facet "first").
          Sergei Kozyrenko made changes -
          Link This issue is duplicated by ICE-4400 [ ICE-4400 ]
          Hide
          Sergei Kozyrenko added a comment -

          This is still an issue with IE 7

          Show
          Sergei Kozyrenko added a comment - This is still an issue with IE 7
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Adnan Durrani [ adnan.durrani ]
          Hide
          Cesar Alba added a comment -

          Hello, i have downloaded version 1.8.0 but the problem still persists with IE 7, using a tree inside a scrollable panelGroup

          Show
          Cesar Alba added a comment - Hello, i have downloaded version 1.8.0 but the problem still persists with IE 7, using a tree inside a scrollable panelGroup
          Ken Fyten made changes -
          Fix Version/s 1.8 [ 10161 ]
          Assignee Priority P3
          Adnan Durrani made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Cannot Reproduce [ 5 ]
          Hide
          Adnan Durrani added a comment -

          Can't reproduce it, please see the attached snapshot. Its working fine with both IE6 and FF.

          Show
          Adnan Durrani added a comment - Can't reproduce it, please see the attached snapshot. Its working fine with both IE6 and FF.
          Adnan Durrani made changes -
          Attachment ICE-3194.jpg [ 11432 ]
          Hide
          Adnan Durrani added a comment -

          Problem already has been fixed.

          Show
          Adnan Durrani added a comment - Problem already has been fixed.
          Adnan Durrani made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 1.8DR#3 [ 10143 ]
          Assignee Priority P3
          Assignee Adnan Durrani [ adnan.durrani ]
          Ken Fyten made changes -
          Link This issue is duplicated by ICE-3459 [ ICE-3459 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-3512 [ ICE-3512 ]
          Sergey Petrov made changes -
          Attachment menuPopupBug.xhtml [ 11073 ]
          Attachment MenuPopupBug.java [ 11074 ]
          Hide
          Sergey Petrov added a comment -

          please use these files to reproduce the bug

          Show
          Sergey Petrov added a comment - please use these files to reproduce the bug
          Sergey Petrov made changes -
          Sergey Petrov made changes -
          Hide
          Sergey Petrov added a comment -

          The menus on this screenshot is result of right-clicking on 'Root' node. Note how the menu divs are shifted down and right.

          Show
          Sergey Petrov added a comment - The menus on this screenshot is result of right-clicking on 'Root' node. Note how the menu divs are shifted down and right.
          Sergey Petrov made changes -
          Field Original Value New Value
          Attachment popup menu clipped in IE6.jpg [ 11070 ]
          Sergey Petrov created issue -

            People

            • Assignee:
              Unassigned
              Reporter:
              Sergey Petrov
            • Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: