ICEfaces
  1. ICEfaces
  2. ICE-6794

menuPopup iceMnuPopVrtSubMenu display:none in IE6 IE7

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: EE-2.0.0.GA, 2.0.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      EE2 Testing

      Description

      Found while testing for EE2 release. With the RichDataGrid component, when you right click on a header, it gives you an ice:menuPopup of the hidden columns. However, ice:menuPopup Sub Menu is not showing up in IE6 or IE7. The div with class iceMnuPopVrtTopSubMenu renders fine, this is the sub menu with class iceMnuPopVrtSubMenu (no 'Top').

      In Firefox the div renders like this:

      <div style="left: 835px; top: 953px;" id="frm:tabs:0:rdg:rdg_showHiddenColumnsMenu:j_idt253_sub" class="iceMnuPopVrtSubMenu">

      In IE7:

      <div class="iceMnuPopVrtSubMenu" id="frm:tabs:0:rdg:rdg_showHiddenColumnsMenu:j_idt253_sub" style="display: none;" visible="function(){var a=update([this],arguments);return __method.apply(null,a);}" toggle="function(){var a=update([this],arguments);return __method.apply(null,a);}" hide="function(){var a=update([this],arguments);return __method.apply(null,a);}" show="function(){var a=update([this],arguments);return __method.apply(null,a);}" remove="function(){var a=update([this],arguments);return __method.apply(null,a);}" update="function(){var a=update([this],arguments);return __method.apply(null,a);}" replace="function(){var a=update([this],arguments);return __method.apply(null,a);}" insert="function(){var a=update([this],arguments);return __method.apply(null,a);}" wrap="function(){var a=update([this],arguments);return __method.apply(null,a);}" inspect="function(){var a=update([this],arguments);return __method.apply(null,a);}" recursivelyCollect="function(){var a=update([this],arguments);return __method.apply(null,a);}" ancestors="function(){var a=update([this],arguments);return __method.apply(null,a);}" descendants="function(){var a=update([this],arguments);return __method.apply(null,a);}" firstDescendant="function(){var a=update([this],arguments);return __method.apply(null,a);}" immediateDescendants="function(){var a=update([this],arguments);return __method.apply(null,a);}" previousSiblings="function(){var a=update([this],arguments);return __method.apply(null,a);}" nextSiblings="function(){var a=update([this],arguments);return __method.apply(null,a);}" siblings="function(){var a=update([this],arguments);return __method.apply(null,a);}" match="function(){var a=update([this],arguments);return __method.apply(null,a);}" up="function(){var a=update([this],arguments);return __method.apply(null,a);}" down="function(){var a=update([this],arguments);return __method.apply(null,a);}" previous="function(){var a=update([this],arguments);return __method.apply(null,a);}" next="function(){var a=update([this],arguments);return __method.apply(null,a);}" select="function(){var a=update([this],arguments);return __method.apply(null,a);}" adjacent="function(){var a=update([this],arguments);return __method.apply(null,a);}" identify="function(){var a=update([this],arguments);return __method.apply(null,a);}" readAttribute="function(){var a=update([this],arguments);return __method.apply(null,a);}" writeAttribute="function(){var a=update([this],arguments);return __method.apply(null,a);}" getHeight="function(){var a=update([this],arguments);return __method.apply(null,a);}" getWidth="function(){var a=update([this],arguments);return __method.apply(null,a);}" classNames="function(){var a=update([this],arguments);return __method.apply(null,a);}" hasClassName="function(){var a=update([this],arguments);return __method.apply(null,a);}" addClassName="function(){var a=update([this],arguments);return __method.apply(null,a);}" removeClassName="function(){var a=update([this],arguments);return __method.apply(null,a);}" toggleClassName="function(){var a=update([this],arguments);return __method.apply(null,a);}" cleanWhitespace="function(){var a=update([this],arguments);return __method.apply(null,a);}" empty="function(){var a=update([this],arguments);return __method.apply(null,a);}" descendantOf="function(){var a=update([this],arguments);return __method.apply(null,a);}" scrollTo="function(){var a=update([this],arguments);return __method.apply(null,a);}" getStyle="function(){var a=update([this],arguments);return __method.apply(null,a);}" getOpacity="function(){var a=update([this],arguments);return __method.apply(null,a);}" setStyle="function(){var a=update([this],arguments);return __method.apply(null,a);}" setOpacity="function(){var a=update([this],arguments);return __method.apply(null,a);}" makePositioned="function(){var a=update([this],arguments);return __method.apply(null,a);}" undoPositioned="function(){var a=update([this],arguments);return __method.apply(null,a);}" makeClipping="function(){var a=update([this],arguments);return __method.apply(null,a);}" undoClipping="function(){var a=update([this],arguments);return __method.apply(null,a);}" clonePosition="function(){var a=update([this],arguments);return __method.apply(null,a);}" getElementsBySelector="function(){var a=update([this],arguments);return __method.apply(null,a);}" childElements="function(){var a=update([this],arguments);return __method.apply(null,a);}" getStorage="function(){var a=update([this],arguments);return __method.apply(null,a);}" store="function(){var a=update([this],arguments);return __method.apply(null,a);}" retrieve="function(){var a=update([this],arguments);return __method.apply(null,a);}" clone="function(){var a=update([this],arguments);return __method.apply(null,a);}" purge="function(){var a=update([this],arguments);return __method.apply(null,a);}" getLayout="function(){var a=update([this],arguments);return __method.apply(null,a);}" measure="function(){var a=update([this],arguments);return __method.apply(null,a);}" getDimensions="function(){var a=update([this],arguments);return __method.apply(null,a);}" getOffsetParent="function(){var a=update([this],arguments);return __method.apply(null,a);}" cumulativeOffset="function(){var a=update([this],arguments);return __method.apply(null,a);}" positionedOffset="function(){var a=update([this],arguments);return __method.apply(null,a);}" cumulativeScrollOffset="function(){var a=update([this],arguments);return __method.apply(null,a);}" viewportOffset="function(){var a=update([this],arguments);return __method.apply(null,a);}" absolutize="function(){var a=update([this],arguments);return __method.apply(null,a);}" relativize="function(){var a=update([this],arguments);return __method.apply(null,a);}" fire="function(){var a=update([this],arguments);return __method.apply(null,a);}" observe="function(){var a=update([this],arguments);return __method.apply(null,a);}" stopObserving="function(){var a=update([this],arguments);return __method.apply(null,a);}" on="function(){var a=update([this],arguments);return __method.apply(null,a);}" childOf="function(){var a=update([this],arguments);return __method.apply(null,a);}" morph="function(){var a=update([this],arguments);return __method.apply(null,a);}" visualEffect="function(){var a=update([this],arguments);return __method.apply(null,a);}" highlight="function(){var a=update([this],arguments);return __method.apply(null,a);}" fade="function(){var a=update([this],arguments);return __method.apply(null,a);}" appear="function(){var a=update([this],arguments);return __method.apply(null,a);}" grow="function(){var a=update([this],arguments);return __method.apply(null,a);}" shrink="function(){var a=update([this],arguments);return __method.apply(null,a);}" fold="function(){var a=update([this],arguments);return __method.apply(null,a);}" blindUp="function(){var a=update([this],arguments);return __method.apply(null,a);}" blindDown="function(){var a=update([this],arguments);return __method.apply(null,a);}" slideUp="function(){var a=update([this],arguments);return __method.apply(null,a);}" slideDown="function(){var a=update([this],arguments);return __method.apply(null,a);}" pulsate="function(){var a=update([this],arguments);return __method.apply(null,a);}" shake="function(){var a=update([this],arguments);return __method.apply(null,a);}" puff="function(){var a=update([this],arguments);return __method.apply(null,a);}" squish="function(){var a=update([this],arguments);return __method.apply(null,a);}" switchOff="function(){var a=update([this],arguments);return __method.apply(null,a);}" dropOut="function(){var a=update([this],arguments);return __method.apply(null,a);}" getInlineOpacity="function(){var a=update([this],arguments);return __method.apply(null,a);}" forceRerendering="function(){var a=update([this],arguments);return __method.apply(null,a);}" setContentZoom="function(){var a=update([this],arguments);return __method.apply(null,a);}" collectTextNodes="function(){var a=update([this],arguments);return __method.apply(null,a);}" collectTextNodesIgnoreClass="function(){var a=update([this],arguments);return __method.apply(null,a);}" getStyles="function(){var a=update([this],arguments);return __method.apply(null,a);}" _extendedByPrototype="function(){}" hasAttribute="function(){var a=update([this],arguments);return __method.apply(null,a);}">

        Activity

        Brad Kroeger created issue -
        Brad Kroeger made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s EE-2.1.0 [ 10262 ]
        Fix Version/s EE-2.0.0.GA [ 10263 ]
        Affects Version/s EE-2.0.0.GA [ 10263 ]
        Affects Version/s 2.0.1 [ 10255 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s EE-2.0.0.GA [ 10263 ]
        Fix Version/s EE-2.1.0 [ 10262 ]
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] yip.ng [ yip.ng ]
        Hide
        yip.ng added a comment -

        All the extra markups are added by IE, probably for debugger use? I normally just ignore them. Nothing to do with the problem at hand.

        Show
        yip.ng added a comment - All the extra markups are added by IE, probably for debugger use? I normally just ignore them. Nothing to do with the problem at hand.
        Hide
        yip.ng added a comment -

        Problem is in Ice.clonePositionIE function added in ICE-4106. It's a complicated function modified from Prototype's Element.clonePosition. Somehow it returns a wrong top coordinate in IE7.

        The display property is also reverted to none from time to time after the positioning. Still can't figure how and why.

        Show
        yip.ng added a comment - Problem is in Ice.clonePositionIE function added in ICE-4106 . It's a complicated function modified from Prototype's Element.clonePosition. Somehow it returns a wrong top coordinate in IE7. The display property is also reverted to none from time to time after the positioning. Still can't figure how and why.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #24438 Wed Apr 20 14:44:51 MDT 2011 yip.ng ICE-6794: menuPopup iceMnuPopVrtSubMenu display:none in IE6 IE7.
        Files Changed
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/compat/core/src/main/javascript/extras/extras.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #24439 Wed Apr 20 14:53:57 MDT 2011 yip.ng ICE-6794: menuPopup iceMnuPopVrtSubMenu display:none in IE6 IE7.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
        yip.ng made changes -
        Attachment screenshot-01.png [ 13126 ]
        Attachment screenshot-02.png [ 13127 ]
        Hide
        yip.ng added a comment - - edited

        Fixed. See screenshots 1 and 2.

        Revision: 24437


        Modified : /icefaces-ee/tags/icefaces-ee-2.0.0.GA/icefaces/compat/core/src/main/javascript/extras/extras.js

        Revision: 24438


        Modified : /icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/compat/core/src/main/javascript/extras/extras.js

        Revision: 24439


        Modified : /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js

        Show
        yip.ng added a comment - - edited Fixed. See screenshots 1 and 2. Revision: 24437 Modified : /icefaces-ee/tags/icefaces-ee-2.0.0.GA/icefaces/compat/core/src/main/javascript/extras/extras.js Revision: 24438 Modified : /icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/compat/core/src/main/javascript/extras/extras.js Revision: 24439 Modified : /icefaces2/trunk/icefaces/compat/core/src/main/javascript/extras/extras.js
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Attachment IncorrectSubMenuVerticalPlacement.png [ 13135 ]
        Ken Fyten made changes -
        Comment [ This fix has caused a layout regression with submenu vertical positioning. If you view the attached IncorrectSubMenuVerticalPlacement.png, you will see that the submenus are now appearing too high vertically vs. the root menuitem that opens them. ]
        Ken Fyten made changes -
        Attachment IncorrectSubMenuVerticalPlacement.png [ 13135 ]
        Ken Fyten made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Assignee Priority P1
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.2 [ 10273 ]
        Affects Version/s 2.0.0 [ 10230 ]
        Affects Version/s EE-2.0.0.GA [ 10263 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: