Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-8406

JS eror when hover over the help icons After M4 update

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 2.2.0-rc1, 2.2
    • Component/s: Accessibility
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-4707Hover over field level help with Dialog
      KULRICE-5895Help text on Hover doesn't display onparts of Route log panel
      KULRICE-2019When any of the HELP icons are clicked no help text is displayed
      KULRICE-9943: Copy icon not appearing on syntax highlighter
      KULRICE-5442disabled header tabs should not highlight on hover
      KULRICE-9915Add line client Javascript runs for over a minute
      KULRICE-7490Highlight collection when hovering on add button
      KULRICE-5639Within the date-picker widget, there are previous and next buttons. In IE, when hovering over these, the images flicker on and off. When clicking, nothing happens.
      KULRICE-8847Role lookups slow when there are over 200 roles
      KULRICE-7462Validation messaging - single checkbox visual treatment/shading missing on hover
    • Rice Module:
      KRAD
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      After M4 update ,When hovered over the Help icons a JS error is thrown(Uncaught TypeError: Cannot read property 'top' of undefined)

      Attaching a screenshot(errorscreenshot.jpg) and bean definitions(helpiconsbeandefinitions.txt) used for reference.

      1. helpiconbeandefinitions.txt
        2 kB
        Hemanth Gajula
      1. errorscreenshot.jpg
        155 kB

        Activity

        Hide
        Hemanth Gajula (Inactive) added a comment - - edited

        Debugging the issue further found that in

        krad_2.2.0-M4.js

        inside the

        function c(E, N, x) {

        var u = N.offset();-->(The offset for the N is undefined)
        var t = parseInt(u.top);-->(u is undefined),Failing at this position and throwing the js error

        Show
        Hemanth Gajula (Inactive) added a comment - - edited Debugging the issue further found that in krad_2.2.0-M4.js inside the function c(E, N, x) { var u = N.offset();-->(The offset for the N is undefined) var t = parseInt(u.top);-->(u is undefined),Failing at this position and throwing the js error
        Hide
        Hemanth Gajula (Inactive) added a comment -

        .. We are overriding the createbubblepopup() function in our js
        .. Previously in krad js the createbubblepopup use to have only one parameter which are the options (Like in original jquery template ".CreateBubblePopup(

        { themePath : ‘…’ }

        )")
        .. But now in krad_2.2.0_m4.js this is been modified to (.CreateBubblePopup(selector,

        { themePath : ‘…’ }

        ))) which we didn't change in our js functions.
        .. So once we turn off our js fucntions we see the bubblepopup.

        Show
        Hemanth Gajula (Inactive) added a comment - .. We are overriding the createbubblepopup() function in our js .. Previously in krad js the createbubblepopup use to have only one parameter which are the options (Like in original jquery template ".CreateBubblePopup( { themePath : ‘…’ } )") .. But now in krad_2.2.0_m4.js this is been modified to (.CreateBubblePopup(selector, { themePath : ‘…’ } ))) which we didn't change in our js functions. .. So once we turn off our js fucntions we see the bubblepopup.
        Hide
        Brian Smith (Inactive) added a comment -

        So this is fixed then?

        Show
        Brian Smith (Inactive) added a comment - So this is fixed then?
        Hide
        Brian Smith (Inactive) added a comment -

        Btw, the changes were made in M4 to fix issues pointed out by myplan if I remember correctly

        Show
        Brian Smith (Inactive) added a comment - Btw, the changes were made in M4 to fix issues pointed out by myplan if I remember correctly
        Hide
        Brian Smith (Inactive) added a comment -

        MyPlan will change their code to accommodate the newly changed function customization done in m4 to the bubblepopup plugin

        Show
        Brian Smith (Inactive) added a comment - MyPlan will change their code to accommodate the newly changed function customization done in m4 to the bubblepopup plugin

          People

          • Assignee:
            Brian Smith (Inactive)
            Reporter:
            Hemanth Gajula (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel