Bug 14253: Acq - notify borrowers popup needs to allow scrolling
authorKatrin Fischer <Katrin.Fischer.83@web.de>
Tue, 9 Jun 2015 01:01:08 +0000 (03:01 +0200)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 22 Jun 2015 20:07:45 +0000 (17:07 -0300)
The 'notify on receiving' patron search on the new order form
in acquisitions didn't allow you to scroll, so there was no
way to select users from the bottom of a longer result list.

To test:
- Create a new order in acquisitions
- On the order form, use the 'Add user' button to open
  the popup
- Perform a patron research with a lot of results
- Verify that with the patch you can scroll, but
  that you couldn't without it

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
without patch: no scroll bar in Firefox 38
with patch: scrolling works fine

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt

index 29018ac..b7f5074 100644 (file)
@@ -165,7 +165,7 @@ $(document).ready(function()
         window.open(
             "/cgi-bin/koha/acqui/add_user_search.pl",
             'UserSearchPopup',
-            'width=740,height=450,toolbar=no,'
+            'width=740, height=450, scrollbars=yes, toolbar=no,'
          );
     }