ICEfaces
  1. ICEfaces
  2. ICE-6812

panelPopup autoCenter mispositions menuPopup within a panelTooltip

    Details

    • Assignee Priority:
      P3
    • Salesforce Case Reference:

      Description

      Specifying autoCenter on the panelPopup causes the nested menuPopup within the panelToolTip to appear in the bottom right hand side of the screen.
      1. screenshot-1.jpg
        21 kB

        Activity

        Tyler Johnson created issue -
        Tyler Johnson made changes -
        Field Original Value New Value
        Attachment screenshot-1.jpg [ 13123 ]
        Tyler Johnson made changes -
        Attachment sc9851.war [ 13124 ]
        Hide
        Tyler Johnson added a comment -

        Test case is intended for deployment on tomcat 6. To activate the tooltip, right click on the inputText components.

        Show
        Tyler Johnson added a comment - Test case is intended for deployment on tomcat 6. To activate the tooltip, right click on the inputText components.
        Tyler Johnson made changes -
        Salesforce Case [5007000000FbnSs]
        Tyler Johnson made changes -
        Salesforce Case Reference 5007000000FbnSsAAJ
        Arran Mccullough made changes -
        Salesforce Case Reference 5007000000FbnSsAAJ 5007000000VHOjyAAH,5007000000FbnSsAAJ
        Arran Mccullough made changes -
        Affects Version/s EE-1.8.2.GA_P07 [ 10672 ]
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s EE-1.8.2.GA_P08 [ 11178 ]
        Assignee Priority P2 [ 10011 ]
        Hide
        Arran Mccullough added a comment -

        I can reproduce similar issue with just a panelPopup and a menuPopup. Test case code can be provided if needed.

        Show
        Arran Mccullough added a comment - I can reproduce similar issue with just a panelPopup and a menuPopup. Test case code can be provided if needed.
        Ken Fyten made changes -
        Assignee Priority P2 [ 10011 ] P1 [ 10010 ]
        Hide
        Arran Mccullough added a comment -

        Attached my current test case.

        Steps:

        • Load welcomeICEfaces.iface
        • Click on the Open button
        • Right click the text on the panelPopup, menuPopup is displayed away from where the right click event takes place.
        Show
        Arran Mccullough added a comment - Attached my current test case. Steps: Load welcomeICEfaces.iface Click on the Open button Right click the text on the panelPopup, menuPopup is displayed away from where the right click event takes place.
        Arran Mccullough made changes -
        Attachment Case12351Example.war [ 16554 ]
        Attachment Case12351Example.zip [ 16555 ]
        Hide
        Arturo Zambrano added a comment -

        Committed fix at revision 38748.

        I could only reproduce the issue on the Case12351Example test case, attached by Arran. It seems like the issue only occurred with modal popups, it had nothing to do with the 'autoCentre' attribute.

        All the coordinates were being set correctly, the only problem was that under certain scenarios, those coordinates would be relative to the container element, so this was fixed by adding 'position:fixed' to the context menu container style attribute.

        Show
        Arturo Zambrano added a comment - Committed fix at revision 38748. I could only reproduce the issue on the Case12351Example test case, attached by Arran. It seems like the issue only occurred with modal popups, it had nothing to do with the 'autoCentre' attribute. All the coordinates were being set correctly, the only problem was that under certain scenarios, those coordinates would be relative to the container element, so this was fixed by adding 'position:fixed' to the context menu container style attribute.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #38748 Thu Nov 07 14:08:52 MST 2013 art.zambrano ICE-6812 fix for menuPopup misposition inside panelpopup
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
        Hide
        Arturo Zambrano added a comment -

        Committed fix to 3.3 EE maintenance branch at revision 38773.

        Show
        Arturo Zambrano added a comment - Committed fix to 3.3 EE maintenance branch at revision 38773.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Arran Mccullough added a comment -

        This fix looks to have caused a regression when displaying the menuPopup in a scrollable browser window. If the window is scrolled at the top, the menuPopup displays in the correct position. If the browser window is scrolled down, the menu gets displayed in an incorrect position. This is seen outside of the panelPopup. I'll attach a modified test case that reproduces the issue.

        Show
        Arran Mccullough added a comment - This fix looks to have caused a regression when displaying the menuPopup in a scrollable browser window. If the window is scrolled at the top, the menuPopup displays in the correct position. If the browser window is scrolled down, the menu gets displayed in an incorrect position. This is seen outside of the panelPopup. I'll attach a modified test case that reproduces the issue.
        Arran Mccullough made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Arran Mccullough made changes -
        Attachment Case12351Example.war [ 16554 ]
        Arran Mccullough made changes -
        Attachment Case12351Example.zip [ 16555 ]
        Hide
        Arran Mccullough added a comment -

        Attached test case that shows regression.

        Steps:

        • Load welcomeICEfaces.iface
        • Resize the browser so a vertical scrollbar is shown.
        • Right click on the Column Two cells, menuPopup is positioned fine.
        • Re-do the above step with the browser scrolled down, position is off.
        Show
        Arran Mccullough added a comment - Attached test case that shows regression. Steps: Load welcomeICEfaces.iface Resize the browser so a vertical scrollbar is shown. Right click on the Column Two cells, menuPopup is positioned fine. Re-do the above step with the browser scrolled down, position is off.
        Arran Mccullough made changes -
        Attachment Case12351Example.war [ 16618 ]
        Attachment Case12351Example.zip [ 16619 ]
        Hide
        Arturo Zambrano added a comment - - edited

        Committed fix to 1.8 trunk at revision 39993 and to 3.3 EE maintenance branch at revision 40001. Since, the menu has been now fixed-positioned, we needed to take into account the vertical scroll when positioning it. So, we now subtract the vertical scroll value from the absolute vertical position on the page, so the menu can be correctly positioned whether the vertical scroll is at the top, at the bottom or in between.

        Show
        Arturo Zambrano added a comment - - edited Committed fix to 1.8 trunk at revision 39993 and to 3.3 EE maintenance branch at revision 40001. Since, the menu has been now fixed-positioned, we needed to take into account the vertical scroll when positioning it. So, we now subtract the vertical scroll value from the absolute vertical position on the page, so the menu can be correctly positioned whether the vertical scroll is at the top, at the bottom or in between.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #39993 Wed Feb 12 11:19:53 MST 2014 art.zambrano ICE-6812 modification to subtract vertical scroll offset when positioning (fixed-positioned) menu
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        Hide
        Liana Munroe added a comment - - edited

        Icefaces1.8 trunk revision# 41326
        Tomcat6, IE7 and 8 only
        Component-showcase, Component-showcase[facelets-enh] and [facelets]
        Menu popup - Error seen when interacting (right clicking) with demo.
        Message: Invalid argument.
        Line: 38
        Char: 3658
        Code: 0
        URI: http://localhost:8080/ICE-3856/xmlhttp/1402676167843/ice-extras.js
        Server error seen:
        SEVERE: java.lang.NullPointerException
        at org.icefaces.application.showcase.view.bean.examples.component.menuPopup.MenuPopupBean.executeMenuEffect(MenuPopupBean.java:85)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.apache.el.parser.AstValue.invoke(AstValue.java:191)
        at org.apache.el.MethodExpressionImpl.invoke(MethodExpressionImpl.java:276)
        at com.sun.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:68)
        at javax.faces.event.MethodExpressionActionListener.processAction(MethodExpressionActionListener.java:99)
        at com.icesoft.faces.component.ContextActionEvent.processListener(ContextActionEvent.java:63)
        at javax.faces.component.UIComponentBase.broadcast(UIComponentBase.java:774)......

        Show
        Liana Munroe added a comment - - edited Icefaces1.8 trunk revision# 41326 Tomcat6, IE7 and 8 only Component-showcase, Component-showcase [facelets-enh] and [facelets] Menu popup - Error seen when interacting (right clicking) with demo. Message: Invalid argument. Line: 38 Char: 3658 Code: 0 URI: http://localhost:8080/ICE-3856/xmlhttp/1402676167843/ice-extras.js Server error seen: SEVERE: java.lang.NullPointerException at org.icefaces.application.showcase.view.bean.examples.component.menuPopup.MenuPopupBean.executeMenuEffect(MenuPopupBean.java:85) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at org.apache.el.parser.AstValue.invoke(AstValue.java:191) at org.apache.el.MethodExpressionImpl.invoke(MethodExpressionImpl.java:276) at com.sun.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:68) at javax.faces.event.MethodExpressionActionListener.processAction(MethodExpressionActionListener.java:99) at com.icesoft.faces.component.ContextActionEvent.processListener(ContextActionEvent.java:63) at javax.faces.component.UIComponentBase.broadcast(UIComponentBase.java:774)......
        Liana Munroe made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Fix Version/s EE-1.8.2.GA_P08 [ 11178 ]
        Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
        Ken Fyten made changes -
        Fix Version/s EE-1.8.2.GA_P08 [ 11178 ]
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        Ken Fyten made changes -
        Assignee Priority P1 [ 10010 ] P3 [ 10012 ]
        Hide
        Arturo Zambrano added a comment -

        Couldn't reproduce the last issue above (1.8 trunk, latest revision). Please verify.

        Show
        Arturo Zambrano added a comment - Couldn't reproduce the last issue above (1.8 trunk, latest revision). Please verify.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 1.8 r44499. Still able to reproduce last issue using IE 7 and 8.

        Show
        Liana Munroe added a comment - Tested with ICEfaces 1.8 r44499. Still able to reproduce last issue using IE 7 and 8.
        Liana Munroe made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #44666 Fri May 08 16:22:06 MDT 2015 art.zambrano ICE-6812 added alternatives for window.pageYOffset, which doesn't exist on IE8
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        Hide
        Arturo Zambrano added a comment -

        r44666: added alternatives for window.pageYOffset, which doesn't exist on IE8.

        Show
        Arturo Zambrano added a comment - r44666: added alternatives for window.pageYOffset, which doesn't exist on IE8.
        Hide
        Liana Munroe added a comment -

        Verified component-showcase issue with IE7 and IE8 resolved, ICEfaces 1.8 r44697, Tomcat 6.

        Show
        Liana Munroe added a comment - Verified component-showcase issue with IE7 and IE8 resolved, ICEfaces 1.8 r44697, Tomcat 6.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #44771 Tue May 26 10:17:16 MDT 2015 art.zambrano ICE-6812 fix to position submenus at the right height when the page has been scrolled down
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        Hide
        Arturo Zambrano added a comment -

        r44771: committed fix to position submenus at the right height when the page has been scrolled down to the 1.8 trunk (it didn't affect the 3.3 codebase).

        An issue I found when working on this was that on 1.8 the submenus were positioned higher when there was scrolling in the viewport. This fix includes the vertical scrolling offset when calculating the position of submenus.

        Show
        Arturo Zambrano added a comment - r44771: committed fix to position submenus at the right height when the page has been scrolled down to the 1.8 trunk (it didn't affect the 3.3 codebase). An issue I found when working on this was that on 1.8 the submenus were positioned higher when there was scrolling in the viewport. This fix includes the vertical scrolling offset when calculating the position of submenus.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces EE-3.3.0.GA_P03 tag r42000 and ICEfaces 1.8 trunk r44773. No issues found.

        Show
        Liana Munroe added a comment - Verified ICEfaces EE-3.3.0.GA_P03 tag r42000 and ICEfaces 1.8 trunk r44773. No issues found.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: