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

Close/Clear Values/Search Buttons too far down on Agenda Editor PeopleFlow ID lookup screen

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.5.1
    • Fix Version/s: 2.5.1
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-6576PeopleFlow Action Id lookup throws NPE
      KULRICE-13204Agenda Editor - Editing a Rule: Hidden ids are being displayed as labels
      KULRICE-6365On rule screen, PeopleFlow ID lookup for action types can pull in bogus parameters
      KULRICE-13320KRMS agenda editor UI handles '>' operator badly
      KULRICE-13321KRMS agenda editor, field labels not rendered for propositions in edit mode
      KULRICE-13203Return to agenda button in agenda inquiry view returns to a blank agenda
      KULRICE-5297Troubleshoot pulling context id value into agenda quickfinder/lookup automatically
      KULRICE-13890Intermittent stacktrace on Agenda Editor context ID lookup
      KULRICE-13205Agenda Editor: Deleting of propositions is not being persisted.
      KULRICE-8364The messages are not getting displayed properly on the Agenda Editor
    • Rice Team:
      Middleware
    • Sprint:
      Middleware 2.5.1 Sprint 3, Middleware 2.5.2 Sprint 1, Middleware 2.5.2 Sprint 2, Middleware 2.5.2 Sprint 3
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes
    • Story Points:
      5

      Description

      Click Create New Agenda.

      Namespace: Kuali Rules Test
      Name: (Name)
      Context: Context1

      Click Add Rule.
      In Action section, select Notify PeopleFlow or Route to PeopleFlow.
      In resulting PeopleFlow ID field, click Lookup.

      (see attachments)

      Also note: There is a surplus of white space in the Existing Rule Name Lookup (Copy from Existing Rule) near the top of this page.

      On smaller screens it appears there are no buttons at all. User must scroll to see them.

        Activity

        Hide
        Steve Edgar (Inactive) added a comment -

        I am seeing the behavior in Safari, Chrome and Firefox.

        The rub is how this line ...

        var height = jQuery(window).height() * .85;

        ... in the function iframeModalResize(), in krad.modal.js, works, depending on if the form is in an iframe.

        If the form is in the Rice Portal, you'll see the problem, as the height of the Lookup Dialog appears to be calculated from the height of the page, as opposed to the "browser viewport". The longer the page, the greater the height of the Lookup Dialog.

        However, if the same form is not in the Rice Portal, the height of the Lookup Dialog is determined from the browser viewport, and not the page. Everything works okay in this scenario.

        The iframeModalResize() appears to be used a lot, and so am not sure if this case is best handled by the KRAD experts. Kristina: can you recommend?

        Show
        Steve Edgar (Inactive) added a comment - I am seeing the behavior in Safari, Chrome and Firefox. The rub is how this line ... var height = jQuery(window).height() * .85; ... in the function iframeModalResize(), in krad.modal.js, works, depending on if the form is in an iframe. If the form is in the Rice Portal, you'll see the problem, as the height of the Lookup Dialog appears to be calculated from the height of the page, as opposed to the "browser viewport". The longer the page, the greater the height of the Lookup Dialog. However, if the same form is not in the Rice Portal, the height of the Lookup Dialog is determined from the browser viewport, and not the page. Everything works okay in this scenario. The iframeModalResize() appears to be used a lot, and so am not sure if this case is best handled by the KRAD experts. Kristina: can you recommend?
        Hide
        Kristina Taylor (Inactive) added a comment -

        I think we've made exceptions for the Rice portal before. See if you can figure out why it's being calculated differently while in an iframe.

        Show
        Kristina Taylor (Inactive) added a comment - I think we've made exceptions for the Rice portal before. See if you can figure out why it's being calculated differently while in an iframe.
        Hide
        Claus Niesen added a comment -

        I checked with Brian and he agreed that there is no good solution to handling the iframe dialogs inside a portal. It's just the cost of using the portal.

        Show
        Claus Niesen added a comment - I checked with Brian and he agreed that there is no good solution to handling the iframe dialogs inside a portal. It's just the cost of using the portal.
        Hide
        Kristina Taylor (Inactive) added a comment -

        Ok, let's close this as a won't fix. At least we have a Jira to point people to for reference.

        Show
        Kristina Taylor (Inactive) added a comment - Ok, let's close this as a won't fix. At least we have a Jira to point people to for reference.
        Hide
        Steve Edgar (Inactive) added a comment -

        Additional background is ... I investigated possible solutions, but did not see one. Collaborated with Claus, who came to the same conclusion. Claus consulted Brian, in case we had missed something.

        Show
        Steve Edgar (Inactive) added a comment - Additional background is ... I investigated possible solutions, but did not see one. Collaborated with Claus, who came to the same conclusion. Claus consulted Brian, in case we had missed something.
        Hide
        Martin Taylor (Inactive) added a comment -

        Closing 2.5.1 Development

        Show
        Martin Taylor (Inactive) added a comment - Closing 2.5.1 Development

          People

          • Assignee:
            Steve Edgar (Inactive)
            Reporter:
            Adam Campbell (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile

                Structure Helper Panel