Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-8731

easyXDM i frame resizing script does not work with IE8 and IE7

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.3
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Application Requirement:
      KC
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      1. Using Internet Explorer 8, click any of the tabs on the main portal page (e.g., Researcher Tab, Central Admin tab, etc.)
      2. Click any of create and search icons or any of the linked menu items on any of the tabs

      Expected Behavior: The system should display the appropriate lookup page, create document page, or other linked page upon the user clicking the respective type of menu item.

      Actual Behavior: The system does not display any lookup page, new document, or any other linked page. The system no longer displays the menu items on the tab. The system persists displaying the selected tab (e.g., Central Admin) as highlighted. See attached screen image.

      Note: Upon the user clicking any of the tabs, including the current highlighted tab, the system displays the main menu areas and items for that tab.

        Attachments

          Activity

          Hide
          gathreya Gayathri Athreya added a comment -

          Adding Bryan in case he wants to pitch in.

          Show
          gathreya Gayathri Athreya added a comment - Adding Bryan in case he wants to pitch in.
          Hide
          cniesen Claus Niesen added a comment -

          I tracked down the problem to the use of window.innerHeight in iframePortletContainer.tag. It's not supported in IE8 or prior. The use of $(window).height() should resolve the issue.

          Show
          cniesen Claus Niesen added a comment - I tracked down the problem to the use of window.innerHeight in iframePortletContainer.tag. It's not supported in IE8 or prior. The use of $(window).height() should resolve the issue.
          Hide
          gathreya Gayathri Athreya added a comment - - edited

          OK. I also saw the following error in the console "SCRIPT87: Invalid argument". easyXDM.js, line 421, character 21". This seems to have to do with including the swf files you mentioned in the original jira. http://easyxdm.net/wp/2011/04/13/microsoft-security-bulletin-ms11-018/ Did you not see it? Am I missing something?

          Show
          gathreya Gayathri Athreya added a comment - - edited OK. I also saw the following error in the console "SCRIPT87: Invalid argument". easyXDM.js, line 421, character 21". This seems to have to do with including the swf files you mentioned in the original jira. http://easyxdm.net/wp/2011/04/13/microsoft-security-bulletin-ms11-018/ Did you not see it? Am I missing something?
          Hide
          cniesen Claus Niesen added a comment -

          If you debug that line

          destination[prop] = source[prop];

          you will notice that a height value of "NaNpx" is being assigned and fails. The "NaN" comes from the window.innerHeight and "px" is appended.

          Show
          cniesen Claus Niesen added a comment - If you debug that line destination[prop] = source[prop]; you will notice that a height value of "NaNpx" is being assigned and fails. The "NaN" comes from the window.innerHeight and "px" is appended.
          Hide
          gathreya Gayathri Athreya added a comment -

          Oh I see, that makes sense. Thanks Claus, let me know when the fix is in and I can test it.

          Show
          gathreya Gayathri Athreya added a comment - Oh I see, that makes sense. Thanks Claus, let me know when the fix is in and I can test it.
          Hide
          cniesen Claus Niesen added a comment -

          The fix is committed.

          Show
          cniesen Claus Niesen added a comment - The fix is committed.
          Hide
          gathreya Gayathri Athreya added a comment -

          Tested with IE8 and IE7, confirmed working.

          Show
          gathreya Gayathri Athreya added a comment - Tested with IE8 and IE7, confirmed working.
          Hide
          cniesen Claus Niesen added a comment -

          Just added a keyword to the title to enable better searching.

          Show
          cniesen Claus Niesen added a comment - Just added a keyword to the title to enable better searching.

            People

            • Assignee:
              cniesen Claus Niesen
              Reporter:
              gathreya Gayathri Athreya
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: