Bug 9059

Summary: Increase size of opac-sendbasket window
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: OPACAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: trivial    
Priority: P5 - low CC: chris, jcamins, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch
[SIGNED-OFF] Bug 9059: Increase size of opac-sendbasket window
Followup removing dependent= parameter
[SIGNED-OFF] Bug 9059: Increase size of opac-sendbasket window
[SIGNED-OFF] Bug 9059: Followup

Description Marcel de Rooy 2012-11-12 11:38:01 UTC
If you need to login first, this window is much too small.
Comment 1 Marcel de Rooy 2012-11-12 11:40:36 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-11-12 17:33:19 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2012-11-14 13:11:50 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2012-11-25 19:05:33 UTC
Created attachment 13654 [details] [review]
[SIGNED-OFF] Bug 9059: Increase size of opac-sendbasket window

Change window settings for opac-sendbasket. If you need to login
first, the window should be much larger.

Typo dependant should be dependent. Note that this setting is
ignored in much browsers. Will submit another patch to cover more occurrences.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 5 Katrin Fischer 2012-11-25 19:05:47 UTC
Created attachment 13655 [details] [review]
[SIGNED-OFF] Bug 9059: Followup

Removes last occurrence of dependent= parameter. See also report 7368.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests.
Tested successfully in Chromium and Firefox in Ubuntu.
Comment 6 Jared Camins-Esakov 2012-11-25 23:43:50 UTC
This patch has been pushed to master.
Comment 7 Chris Cormack 2012-11-26 07:13:41 UTC
Pushed to 3.10.x, will be in 3.8.x
Comment 8 Chris Cormack 2012-11-26 07:14:09 UTC
Heh, that should read, will be in 3.10.1